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

chore(ci): replace deprecated toolchain action #2205

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

The-personified-devil
Copy link
Collaborator

This would also allow us to directly specify a version of rust that we want

@The-personified-devil The-personified-devil force-pushed the actions-update branch 2 times, most recently from 863858d to af4be71 Compare June 21, 2024 20:40
@zmerp
Copy link
Member

zmerp commented Jun 26, 2024

Why is this PR draft?

override: true
- uses: Swatinem/rust-cache@v1
- uses: actions/checkout@v4
- uses: dtolnay/[email protected]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say not to download the toolchain for the current msrv. It's going to be done by xtask and I want the version to be pulled dynamically from the root Cargo.toml.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea this was only an attempt to fix cargo-msrv

@The-personified-devil
Copy link
Collaborator Author

Why is this PR draft?

There's complete replacements that also solve the clippy issue, so I want to use them instead, see discord for reference

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.

CI: Update actions toolchain
2 participants