Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README: Fix links to EEP 0 #54

Merged
merged 1 commit into from
Sep 28, 2023
Merged

README: Fix links to EEP 0 #54

merged 1 commit into from
Sep 28, 2023

Conversation

juhlig
Copy link
Contributor

@juhlig juhlig commented Sep 28, 2023

No description provided.

@garazdawi garazdawi merged commit c98d301 into erlang:master Sep 28, 2023
1 check passed
Copy link
Contributor

@RaimoNiskanen RaimoNiskanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old link points to our download server, on which we apparently stopped serving this directory.
So this link redirects to the web server at www.erlang.org that for the /eeps/ directory auto appends .html. I think we should not rely on that behaviour so the new link should be to http://erlang.org/eeps/eep-0000.html.
Then the web server should stop removing .html from the links it translates within EEP 0.
I think it should not be hidden that there is the source format .md that is translated to the view format .html, and that the web server should serve both, explicitly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants