style: address comments

.github/workflows/build.yml

@@ -23,15 +23,15 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: format
       run: cargo fmt -- --check
 
   clippy:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Configure Cargo cache
       uses: actions/cache@v3
       with:
@@ -50,14 +50,14 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Build
         run: cargo build --verbose
 
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Test
         run: cargo test --verbose
 
@@ -68,7 +68,7 @@ jobs:
       options: --security-opt seccomp=unconfined
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Generate code coverage
         run: |
           cargo +nightly tarpaulin --verbose --all-features --workspace --timeout 120 --out Lcov --output-dir ./coverage

CHANGELOG.md

@@ -8,10 +8,14 @@ and this project adheres to [Conventional Commits](https://www.conventionalcommi
 ## [Unreleased]
 
 ## [0.4.1] - 2025-01-29
-- Applied more modern rust code styling, rustfmt and cargo clippy.
-- Replaced test bencher with criterion.
-- Moved non release dependencies to `dev-dependencies`.
-- `cargo update`
+
+### Changed
+- Apply more modern Rust code styling, including `rustfmt` and `cargo clippy`.
+- Replace test bencher with `criterion`.
+- Move non-release dependencies to `dev-dependencies`.
+
+### Updated
+- Run `cargo update` to update dependencies.
 
 ## [0.4.0] - 2023-11-01
 

Cargo.toml

@@ -37,4 +37,4 @@ harness = false
 
 [[bench]]
 name = "formatters"
-harness = false
+harness = false

rustfmt.toml

@@ -2,4 +2,4 @@ reorder_imports = false
 reorder_modules = false
 
 use_field_init_shorthand = true
-use_try_shorthand = true
+use_try_shorthand = true

src/maze/formatters/game_map.rs

@@ -427,39 +427,25 @@ mod tests {
     #[test]
     fn possible_start_and_goal_positions() {
         let formatter = GameMap::new().with_start_goal();
+        #[rustfmt::skip]
         let map = vec![
-            '#', '#', '#', '#', '#', '#', '#', '#', '#', '#', '.', '.', '.', '#', '.', '.', '.',
-            '#', '#', '#', '#', '.', '#', '.', '#', '.', '#', '#', '.', '.', '.', '#', '.', '#',
-            '.', '#', '#', '.', '#', '#', '#', '.', '#', '.', '#', '#', '.', '.', '.', '.', '.',
-            '#', '.', '#', '#', '#', '#', '#', '#', '#', '#', '.', '#', '#', '.', '.', '.', '.',
-            '.', '.', '.', '#', '#', '#', '#', '#', '#', '#', '#', '#', '#',
+            '#', '#', '#', '#', '#', '#', '#', '#', '#', 
+            '#', '.', '.', '.', '#', '.', '.', '.', '#', 
+            '#', '#', '#', '.', '#', '.', '#', '.', '#',
+            '#', '.', '.', '.', '#', '.', '#', '.', '#', 
+            '#', '.', '#', '#', '#', '.', '#', '.', '#', 
+            '#', '.', '.', '.', '.', '.', '#', '.', '#', 
+            '#', '#', '#', '#', '#', '#', '#', '.', '#', 
+            '#', '.', '.', '.', '.', '.', '.', '.', '#', 
+            '#', '#', '#', '#', '#', '#', '#', '#', '#',
         ];
+        #[rustfmt::skip]
         let positions = vec![
-            (0, 1),
-            (0, 2),
-            (0, 3),
-            (0, 5),
-            (0, 6),
-            (0, 7),
-            (1, 0),
-            (1, 8),
-            (2, 8),
-            (3, 0),
-            (3, 8),
-            (4, 0),
-            (4, 8),
-            (5, 0),
-            (5, 8),
-            (6, 8),
-            (7, 0),
-            (7, 8),
-            (8, 1),
-            (8, 2),
-            (8, 3),
-            (8, 4),
-            (8, 5),
-            (8, 6),
-            (8, 7),
+            (0, 1), (0, 2), (0, 3), (0, 5), (0, 6),
+            (0, 7), (1, 0), (1, 8), (2, 8), (3, 0),
+            (3, 8), (4, 0), (4, 8), (5, 0), (5, 8),
+            (6, 8), (7, 0), (7, 8), (8, 1), (8, 2),
+            (8, 3), (8, 4), (8, 5), (8, 6), (8, 7),
         ];
         let result = formatter.iter_possible_start_and_goal_positions(&map, 9, 9);
         assert_eq!(positions, result.collect::<Vec<_>>());