-
Notifications
You must be signed in to change notification settings - Fork 12
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 async-nats #66
Comments
@dejanb we still can see the warning I think that is because of cyclic dependency with trustification:
either way we need to fix here first 👍 |
Yeah, that makes sense. We need to update traustification as well. Maybe it's time to cut the first "release"? |
as an upstream-dependency-for-trustification I would say yes 👍 |
we have breaking changes when updating |
* Details trustification#66 (comment) * Also simplifies ci a bit removing unused configs
I would update everything we can. Can you figure out how big is the breaking change for reqwest? Is there newer 0.11.x? |
good approach +1 I'll take a closer look I remember a good amount of compilation errors etc. |
@dejanb I suspect that is messing with the other dependency https://crates.io/crates/graphql_client/0.14.0/dependencies maybe better to wait for new graphql_client version , makes sense ? |
this is the last outdated all the others got updated by the PR |
* Details #66 (comment) * Also simplifies ci a bit removing unused configs
trustification dependabot vulnerability alerts
https://gist.github.com/helio-frota/da4c1c984d12cc458f5c58e8b9201fc6
The text was updated successfully, but these errors were encountered: