Skip to content

Commit

Permalink
Update actions (#10)
Browse files Browse the repository at this point in the history
* Update actions

Fixes node@16 warnings.

* Ignore pedantic clippy lint that pushes for let-else

Our MSRV is 1.58 and let-else is stable in 1.65.

This can also use `let _ = if let ... else {}` but that looks objectively
worse than the match.
  • Loading branch information
parasyte committed Feb 10, 2024
1 parent f7dbe9f commit 2c1c41b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
- 1.58.0
steps:
- name: Checkout sources
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Install toolchain
uses: dtolnay/rust-toolchain@master
with:
Expand All @@ -33,7 +33,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout sources
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Install toolchain
uses: dtolnay/rust-toolchain@master
with:
Expand All @@ -44,7 +44,7 @@ jobs:
with:
shared-key: common
- name: Install cargo-machete
uses: baptiste0928/cargo-install@v2
uses: baptiste0928/cargo-install@v3
with:
crate: cargo-machete
- name: Cargo fmt
Expand All @@ -68,7 +68,7 @@ jobs:
- 1.58.0
steps:
- name: Checkout sources
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Install toolchain
uses: dtolnay/rust-toolchain@master
with:
Expand Down
2 changes: 2 additions & 0 deletions src/ty.rs
Original file line number Diff line number Diff line change
Expand Up @@ -243,6 +243,8 @@ mod tests {
let expected = Ident::new("foo", Span::call_site());

let group = input.try_group().unwrap();
// TODO: This can be replaced with `let else` after MSRV 1.65
#[allow(clippy::single_match_else)]
let tree = match group.stream().into_iter().next() {
Some(TokenTree::Ident(ident)) => ident,
_ => panic!(),
Expand Down

0 comments on commit 2c1c41b

Please sign in to comment.