-
Notifications
You must be signed in to change notification settings - Fork 758
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
fix: msrv CI for object_store #5866
Conversation
f635841
to
83994a0
Compare
83994a0
to
6897382
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much @korowa -- I kicked off the CI and if it passes I plan to merge this PR in
@@ -119,17 +119,20 @@ jobs: | |||
run: cargo update -p ahash --precise 0.8.7 | |||
- name: Check arrow | |||
working-directory: arrow | |||
run: cargo msrv verify | |||
run: cargo msrv --log-target stdout verify |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 for logging to stdout
- name: Downgrade object_store dependencies | ||
working-directory: object_store | ||
# Necessary because 1.30.0 updates MSRV to 1.63 | ||
run: cargo update -p tokio --precise 1.29.1 | ||
# Necessary because tokio 1.30.0 updates MSRV to 1.63 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
Thanks again @korowa 🙏 |
Which issue does this PR close?
Closes #5864.
Rationale for this change
Fixing MSRV CI for object store
What changes are included in this PR?
url
crate while CI explicitly (it seems to be the way to handle such cases)Are there any user-facing changes?
No