ci: bump dtolnay/rust-toolchain from 1.95.0 to 1.120.0 - #277
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [dtolnay/rust-toolchain](https://github.com/dtolnay/rust-toolchain) from 1.95.0 to 1.120.0. - [Release notes](https://github.com/dtolnay/rust-toolchain/releases) - [Commits](dtolnay/rust-toolchain@1.95.0...1.120.0) --- updated-dependencies: - dependency-name: dtolnay/rust-toolchain dependency-version: 1.120.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
Pull Request Overview
This PR should not be merged in its current state. The update to the minimal-features job is problematic because it is designed to be a static guard for the project's Minimum Supported Rust Version (MSRV). Allowing an automated tool to bump this version defeats the purpose of testing against the oldest supported toolchain. Furthermore, the syntax used for the GitHub Action version is invalid for this specific action, which will lead to immediate CI failures. Codacy analysis indicates the project is currently up to standards, but this change would introduce a functional regression in the CI pipeline.
About this PR
- The 'minimal-features' job is documented as 'Pinned to the declared MSRV' to prevent regressions on the minimum supported version. Automatically bumping this via Dependabot defeats the intentionality of this guard. This update should only be performed manually when the project's official
rust-versioninCargo.tomlis also being updated.
Test suggestions
- Verify that the project successfully compiles and passes tests on the new MSRV (1.120.0).
- Confirm that 1.120.0 is the intended new minimum version and that the change is reflected in the project's Cargo.toml (rust-version field).
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that the project successfully compiles and passes tests on the new MSRV (1.120.0).
2. Confirm that 1.120.0 is the intended new minimum version and that the change is reflected in the project's Cargo.toml (rust-version field).
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
|
|
||
| - name: Install Rust toolchain | ||
| uses: dtolnay/rust-toolchain@1.95.0 | ||
| uses: dtolnay/rust-toolchain@1.120.0 |
There was a problem hiding this comment.
🔴 HIGH RISK
This update is problematic for several reasons:
- Invalid Action Reference: The
dtolnay/rust-toolchainaction typically does not use semantic version tags like@1.120.0for the action itself (usually@masteror@stable). Using an invalid reference will cause the GitHub Action to fail to initialize. To pin the toolchain version, use thetoolchaininput parameter instead. - MSRV Guard Violation: This job is a guard for the project's MSRV. Bumping it raises the minimum requirement without a corresponding update to
Cargo.toml, meaning you are no longer testing your actual minimum supported version. - Security Risk: Pinning to a mutable tag (1.120.0) is a security risk. It is a best practice to pin actions to a specific 40-character commit SHA to prevent supply chain attacks.
Bumps dtolnay/rust-toolchain from 1.95.0 to 1.120.0.
Commits
237493ctoolchain: 1.120.0Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)