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

Update rust-toolchain to toml format #3323

Merged
merged 1 commit into from
Apr 21, 2023

Conversation

nick-mobilecoin
Copy link
Collaborator

@nick-mobilecoin nick-mobilecoin commented Apr 21, 2023

Previously the rust-toolchain file was using the legacy one line toolchain version format from https://rust-lang.github.io/rustup/overrides.html#the-toolchain-file. Now the rust-toolchain file uses toml format and updates the nightly version to the last nightly that contained 1.68.

Motivation

Dependabot has been updated to only read the toml format of the rust-toolchain file.

Keeping the changes targeted on the dependabot fix, the rust-toolchain file was not renamed to the newer rust-toolchain.toml.

The nightly version was bumped to the last nightly that was 1.68. This was to bring in the https/sparse crates.io logic that came in 1.68 which dependabot relies on for 1.68 toolchain versions. This was only moved a little to minimize other clippy and format changes that might be needed.

For reference the nightly dates and version transitions:

Date rustc version
2023-01-04 1.68.0-nightly
2023-01-22 1.68.0-nightly
2023-01-23 1.69.0-nightly
2023-03-04 1.69.0-nightly
2023-03-05 1.70.0-nightly
2023-04-15 1.70.0-nightly
2023-04-16 1.71.0-nightly
2023-04-21 1.71.0-nightly

2023-01-04 was the previous nightly version used.

Previously the rust-toolchain file was using the legacy one line
toolchain version format from
<https://rust-lang.github.io/rustup/overrides.html#the-toolchain-file>.
Now the rust-toolchain file uses toml format and updates the nightly
version to the last nightly that contained 1.68
@nick-mobilecoin
Copy link
Collaborator Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@nick-mobilecoin nick-mobilecoin self-assigned this Apr 21, 2023
@nick-mobilecoin nick-mobilecoin requested review from a team, samdealy and varsha888 and removed request for a team April 21, 2023 15:06
@nick-mobilecoin
Copy link
Collaborator Author

you can see a dependabot run using the same toolchain file at https://github.com/nick-mobilecoin/rust-buf-build/network/updates

@nick-mobilecoin nick-mobilecoin merged commit b23a1b7 into master Apr 21, 2023
@nick-mobilecoin nick-mobilecoin deleted the nick/rust-toolchain-for-dependabot branch April 21, 2023 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants