Skip to content
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

maint: update brod dep to 4.0 #146

Merged
merged 2 commits into from
Jul 7, 2024

Conversation

deepfryed
Copy link
Contributor

Updates brod dep to 4.0, should fix #144

@josevalim
Copy link
Member

Can you please run mix deps.unlock —unused as well?

mix.exs Outdated Show resolved Hide resolved
@josevalim josevalim merged commit b2dd8c3 into dashbitco:main Jul 7, 2024
4 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@deepfryed deepfryed deleted the deps/update-brod-4.0 branch July 7, 2024 01:00
@deepfryed
Copy link
Contributor Author

thanks, appreciate the quick review / feedback cycle.

@slashmili
Copy link
Collaborator

since we support both brod versions, I'd say let's push it to hex.

For the record the integration test failed on main but passed after I re-ran it today!

@esambo
Copy link

esambo commented Jul 30, 2024

BTW, it looks like 93fdc70 that has tag v0.4.4 hasn't been merged to master. GitHub shows an error with:

This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

@josevalim
Copy link
Member

Pushed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fails to compile on otp 27
4 participants