diff --git a/.github/workflows/pre-commit.yml b/.github/workflows/pre-commit.yml index 3d04055..491030a 100644 --- a/.github/workflows/pre-commit.yml +++ b/.github/workflows/pre-commit.yml @@ -24,4 +24,6 @@ jobs: steps: - uses: actions/checkout@v4 - uses: actions/setup-python@v5 + with: + python-version: '3.x' - uses: pre-commit/action@v3.0.1 diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index fe1925c..ecda57f 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -45,7 +45,7 @@ We ask that commits are atomic, meaning they are complete and have a single resp PRs should tell a cohesive story, with test and refactor commits that keep the fix or feature commits simple and clear. -Specifically, we would encouage +Specifically, we would encourage - File renames be isolated into their own commit - Add tests in a commit before their feature or fix, showing the current behavior. The diff for the feature/fix commit will then show how the behavior changed, diff --git a/Cargo.toml b/Cargo.toml index d84d9e9..37f5d65 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -80,6 +80,7 @@ string_lit_as_bytes = "warn" string_to_string = "warn" todo = "warn" trait_duplication_in_bounds = "warn" +uninlined_format_args = "warn" verbose_file_reads = "warn" wildcard_imports = "warn" zero_sized_map_values = "warn" diff --git a/src/assert.rs b/src/assert.rs index ef3a8a6..684f582 100644 --- a/src/assert.rs +++ b/src/assert.rs @@ -470,7 +470,7 @@ mod test { fn into_path_from_pred() { let pred = convert_path(predicate::eq(path::Path::new("hello.md"))); let case = pred.find_case(false, path::Path::new("hello.md")); - println!("Failing case: {:?}", case); + println!("Failing case: {case:?}"); assert!(case.is_none()); } @@ -478,7 +478,7 @@ mod test { fn into_path_from_bytes() { let pred = convert_path(b"hello\n" as &[u8]); let case = pred.find_case(false, path::Path::new("tests/fixture/hello.txt")); - println!("Failing case: {:?}", case); + println!("Failing case: {case:?}"); assert!(case.is_none()); } @@ -486,7 +486,7 @@ mod test { fn into_path_from_str() { let pred = convert_path("hello\n"); let case = pred.find_case(false, path::Path::new("tests/fixture/hello.txt")); - println!("Failing case: {:?}", case); + println!("Failing case: {case:?}"); assert!(case.is_none()); } }