Move to dtolnay/toolchain #211
Conversation
kayabaNerve
added
improvement
Introduces malleability by referring to HEAD of dtolnay, yet GHA errored on the prior syntax.
|
Tests take notably longer. Will merge this and create issue for cache. |
|
pstt fyi I forked https://github.com/rustsec/audit-check - PR's welcome There was a discussion that we might host some simple version that doesn't require NodeJS |
|
Sorry, yet I'm not entirely sure where/what you forked. As for a version which doesn't require NodeJS, that sounds great just as we're discussing minimizing bloat. I don't think it's worthwhile enough for us to move though. If it also handled toolchain caching intelligently, that'd be pleasant though (since we use stable, nightly, additional targets, various components, update nightly monthly...). Would love to not hammer Rust's servers while also only downloading relevant pieces from the cache (while simultaneously not getting stuck on an old version of nightly from the cache). Feel free to clarify, as I wouldn't necessarily mind poking around :) I'd also love if an org handled it, not an individual... |
|
We forked https://github.com/actions-rs/audit-check/ into https://github.com/rustsec/audit-check I guess we need to address this/these deprecation/s there now too in our fork |
|
Ah, got it, thanks. We actually don't currently use audit-check. We use Forking the existing action, and merging the open PRs on it, would be seamless to everyone else. RustSec also should be a trusted organization. I'd love to see y'all step up to handle it. |
actions-rs/toolchain is unmaintained (actions-rs/toolchain#216).
actions-rs will stop working in a few months thanks to the deprecation of
set-outputand NodeJS 12 (actions-rs/toolchain#219 and actions-rs/toolchain#221, respectively).This moves to https://github.com/dtolnay/rust-toolchain, which is maintained, and deletes cached-rust per commentary in dtolnay/rust-toolchain#26. We may later want to add https://github.com/Swatinem/rust-cache/ in.