-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update Elixir/OTP and CI #90
Conversation
Map key ordering changes in OTP 26. See https://github.com/elixir-lang/elixir/releases/tag/v1.14.4
{":// should fail", []}, | ||
{"://", []}, | ||
{"ftps://foo.bar/", []}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe move them to valid_urls check so that it's explicit they are accepted now ? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done :)
- run: mix coveralls.github | ||
- run: mix dialyzer | ||
- name: Set up Earthly | ||
uses: earthly/actions-setup@v1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I was keen on trying out earthly, looks great <3
Overall looks good :) If you can maybe just move the |
The minimum version of Elixir is bumped up to 1.13,
so that it is guaranteed that
URI.new/1
will be available.Maintaining support for Elixir 1.9 would have meant there being no way
of supporting OTP 26 for all Elixir versions (you either need
:http_uri.parse/1
or
URI.new/1
).OTP 26 support also required a change to the String validator's doctest.
Earthly is introduced in order to run the same versions in dev as in CI.
Currently,
mix credo --strict
will not run as there's a TODO in social security.