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

bump rustler_precompiled #656

Closed
wants to merge 2 commits into from
Closed

bump rustler_precompiled #656

wants to merge 2 commits into from

Conversation

DnOberon
Copy link

@DnOberon DnOberon commented Oct 19, 2024

Need to bump rustler_precomplied so the CA_CERTS env var can be used for HTTP requests.

@tessi - this update is needed so those behind corporate firewalls can use :rustler_precompiled.

@tessi
Copy link
Owner

tessi commented Oct 23, 2024

@DnOberon thanks for your PR! 💚
Happy to bump rustler, can you please run mix deps.get to update the lockfile?

@DnOberon
Copy link
Author

DnOberon commented Oct 23, 2024 via email

@DnOberon
Copy link
Author

@tessi done!

@DnOberon
Copy link
Author

I'm not sure why the checks are failing @tessi :\

@tessi
Copy link
Owner

tessi commented Nov 8, 2024

hmm, weird. I have the same failures on main. will investigate :)

@tessi
Copy link
Owner

tessi commented Nov 8, 2024

ahh, it's some multivalue bug - and that feature always was a bit shaky. I'll disable it in main and then do a run around updating all the deps (including this one)

@tessi
Copy link
Owner

tessi commented Nov 8, 2024

@DnOberon I updated rustler_precompiled and fixed those error (by removing the tests 😅 ). Ended up updating all the things and releasing a new version v0.9.2

Thanks again for opening this PR and being patient with me. It took me until I had a day off work to find the time :)

@DnOberon
Copy link
Author

DnOberon commented Nov 8, 2024

Thank you :)

@DnOberon DnOberon closed this Nov 8, 2024
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.

2 participants