Upgrade: elixir 1.17 otp 27 node 22 #699
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I tried to review (manually test) the currently open dependabot PR and found myself unable to run the project on an M2 mac.
So I invested about an hour to try and upgrade the thing to the latest and greatest
Outside of that, i had to manually bump
httpoisonashttpoison -> hackney -> ssl_verify_funrefused to compile.I also did a
mix hex.update --allFinally, I bumped everything in the
Dockerfileto and then tried to deploy on a fly machine of my own. I found the min required version for dart_sass went up to 1.55 on linux, so since i was already bumping that, I went with the latest version.And then after the PR failed, I realized I also had to upgrade credo cause it wouldn't run
After setting the necessary secrets, the deployment worked and everything seems to be in order. I took my own app down after, but I can put it back up if anyone wants to manually test.