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 Axum to 0.7.x #519

Closed
wants to merge 2 commits into from
Closed

Bump Axum to 0.7.x #519

wants to merge 2 commits into from

Conversation

jamil7
Copy link
Contributor

@jamil7 jamil7 commented Jan 20, 2024

No description provided.

@buehler
Copy link
Collaborator

buehler commented Jan 22, 2024

Hey @jamil7
Thanks for the contribution! I tried to merge it but sadly the test got rejected.

@jamil7
Copy link
Contributor Author

jamil7 commented Jan 22, 2024

@buehler Should be fixed now. Btw, is there any reason for the CI not to run without approval?

@buehler
Copy link
Collaborator

buehler commented Jan 22, 2024

Hey @jamil7
Yes, this is a security feature for github organizations. As far as I know it is there to prevent leakage of secrets or something. It is only in place for first time contributors, not to worry.

@buehler buehler enabled auto-merge (squash) January 22, 2024 20:29
buehler added a commit that referenced this pull request Jan 23, 2024
@buehler buehler closed this in #521 Jan 23, 2024
buehler added a commit that referenced this pull request Jan 23, 2024
* feat: update general dependencies and axum to v0.7

Closes #514.
Closes #519.
Closes #518.

* ignore marvin attack for now. there is no fix

* fix doctest
auto-merge was automatically disabled January 23, 2024 10:44

Pull request was closed

Copy link

🎉 This issue has been resolved in version 3.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants