CaliDog/certstream-server

Broken link to "supervisor configurations" in README

Closed this issue · 3 comments

AKarbas commented

Hi!

The Link to "supervisor configurations" is broken.
It's on line 53 of the readme: Link.

Cheers.

Synse commented

lib/supervisor.ex was removed in 354138e.

The readme should probably be updated to point at lib/application.ex.

AKarbas commented

I was about to create a PR changing the link, but I really don't find the sentence "is essentially entirely outlined in the supervisor configuration." in the readme true.

Maybe it's because I don't know Elixir, but that link does not help me understand the internals!
The drawn schema is pretty simple to understand tho.

Why not remove the first paragraph?

  • 👍1
AKarbas commented

As is obvious, created a PR.
The end result makes the most sense to me.