Skip to content

Conversation

garazdawi
Copy link
Contributor

Turns out that there actually was a test for this, but as the codepath was Elixir 1.18 only it was never tested in CI. So this PR adds Elixir 1.18 to the CI matrix and fixes the fun problem.

Fixes regression in #1980

@josevalim josevalim merged commit c46b012 into elixir-lang:main Jan 28, 2025
5 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@garazdawi
Copy link
Contributor Author

@josevalim When you have the time it would be great with another rc with this change. I've now built ex_doc locally and everything works as it should.

@josevalim
Copy link
Member

RC2 is out. Once you are all happy with it let me know and I can ship final!

@garazdawi
Copy link
Contributor Author

Thanks! I will test it now.

@garazdawi
Copy link
Contributor Author

Testing takes a while, but now it is all done and working as it should. Thanks!

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.

2 participants