-
Notifications
You must be signed in to change notification settings - Fork 17
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 deps and fix 2 warnings #34
base: main
Are you sure you want to change the base?
Update deps and fix 2 warnings #34
Conversation
@vanjacosic thanks for this! I think we need to bump the minimum Elixir version for this to work. Can you take a look? |
@philss Done 😊
|
elixir: 1.12.3 | ||
otp: 24.3.4 | ||
elixir: 1.13.4 | ||
otp: 24.3.4.17 |
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.
@vanjacosic Can we support Elixir 1.17.x and Erlang OTP 27.x in the CI pipeline?
Updated dependencies and ran tests ✅
Fixed the following compile warning re. deprecation of
push_redirect/2
:Fixed the following Dialyzer warning about an 'unknown type':
Seems to have been an old typo that was never caught.
I ran tests and
mix dev
multiple times and everything seems to work, but please double check 😅