Skip to content

Commit

Permalink
Merge pull request #362 from matthiasbeyer/clippy-on-msrv
Browse files Browse the repository at this point in the history
Run clippy only on MSRV
  • Loading branch information
matthiasbeyer committed Aug 2, 2022
2 parents af537b7 + 518a3ca commit ab556a9
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
8 changes: 1 addition & 7 deletions .github/workflows/msrv.yml
Expand Up @@ -111,20 +111,14 @@ jobs:
needs: [check]
name: Clippy
runs-on: ubuntu-latest
strategy:
matrix:
rust:
- stable
- beta
- nightly
steps:
- name: Checkout sources
uses: actions/checkout@v3.0.2

- name: Install toolchain
uses: actions-rs/toolchain@v1
with:
toolchain: ${{ matrix.rust }}
toolchain: 1.56.1
override: true
components: clippy

Expand Down
4 changes: 3 additions & 1 deletion tests/env.rs
Expand Up @@ -191,7 +191,9 @@ fn test_parse_float() {

// can't use `matches!` because of float value
match config {
TestFloatEnum::Float(TestFloat { float_val }) => assert_eq!(float_val, 42.3),
TestFloatEnum::Float(TestFloat { float_val }) => {
assert!(float_cmp::approx_eq!(f64, float_val, 42.3))
}
}
})
}
Expand Down
2 changes: 1 addition & 1 deletion tests/file_json5.rs
Expand Up @@ -84,7 +84,7 @@ fn test_error_parse() {
assert_eq!(
res.unwrap_err().to_string(),
format!(
" --> 2:7\n |\n2 | ok: true\n | ^---\n |\n = expected null in {}",
" --> 2:7\n |\n2 | ok: true\n | ^---\n |\n = expected null in {}",
path_with_extension.display()
)
);
Expand Down

0 comments on commit ab556a9

Please sign in to comment.