Force rust toolchain for web-client NPM build #3040
Conversation
sisou
force-pushed
the
soeren/npm-ci
branch
2 times, most recently
from
8c35560 to
1eba829
Compare
|
Manual CI run with this branch passed: https://github.com/nimiq/core-rs-albatross/actions/runs/11801453470 |
And remove unnecessary steps and deviations from the default. [skip ci]
| @@ -14,19 +14,15 @@ jobs: | |||
| - uses: actions/checkout@v4 | |||
| - uses: dtolnay/rust-toolchain@stable | |||
There was a problem hiding this comment.
When you want to use a different toolchain other than the current stable, you are supposed to get it by: dtolnay/[email protected].
The selection of Rust toolchain is made based on the particular @ rev of this Action being requested. For example "dtolnay/rust-toolchain@nightly" pulls in the nightly Rust toolchain, while "dtolnay/[email protected]" pulls in 1.42.0.
Important: the default is to match the @ rev as described above. When passing an explicit toolchain as an input instead of @ rev, you'll want to use "dtolnay/rust-toolchain@master" as the revision of the action.
There was a problem hiding this comment.
Welp, it still works. If they want to enforce that, they should prevent the other ways from working :D
This fixes #3022.
Pull request checklist
All tests pass. The project builds and runs.clippyandrustfmtwarnings.