Skip to content

Update Broken Links / Migrate to HTTPS#1828

Merged
josevalim merged 2 commits into
elixir-lang:mainfrom
maennchen:jm/hexdocs-links
Jun 1, 2026
Merged

Update Broken Links / Migrate to HTTPS#1828
josevalim merged 2 commits into
elixir-lang:mainfrom
maennchen:jm/hexdocs-links

Conversation

@maennchen
Copy link
Copy Markdown
Member

  • Sets up Link Checking CI
  • Changes HTTP URLs to HTTPS
  • Updates links

I extensively used AI to do the URL replacement.

@maennchen maennchen force-pushed the jm/hexdocs-links branch 8 times, most recently from a749f65 to 45c3d3b Compare June 1, 2026 11:20
@maennchen
Copy link
Copy Markdown
Member Author

Sorry for the push noise, there was some rate limit issues at first. But now that it has a cache, it works.

Comment thread _posts/2012-12-04-elixir-v0-7-2-released.markdown Outdated
@maennchen maennchen force-pushed the jm/hexdocs-links branch 2 times, most recently from 3b40be1 to 55ec440 Compare June 1, 2026 11:36
@maennchen
Copy link
Copy Markdown
Member Author

Ready for review now.

@josevalim josevalim merged commit b5062b5 into elixir-lang:main Jun 1, 2026
@josevalim
Copy link
Copy Markdown
Member

💚 💙 💜 💛 ❤️

@maennchen maennchen deleted the jm/hexdocs-links branch June 1, 2026 11:59
@9p4
Copy link
Copy Markdown

9p4 commented Jun 2, 2026

This PR was incorrect, as it replaced the namespace in atom.xml from http to https, but the specification requires the link be http, causing the RSS feed on the website to break.

https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Felixir-lang.org%2Fatom.xml

Comment thread atom.xml
---
<?xml version="1.0" encoding="utf-8"?>
<feed xmlns="http://www.w3.org/2005/Atom">
<feed xmlns="https://www.w3.org/2005/Atom">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This specifically should not have been changed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you, fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants