Merge pull request #1375 from rust-ndarray/rustfmt2

Add rustformat config and CI

Author: bluss

.github/workflows/ci.yaml

@@ -26,6 +26,22 @@ jobs:
           components: clippy
       - uses: Swatinem/rust-cache@v2
       - run: cargo clippy --features docs
+
+  format:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rust:
+          - nightly
+    name: format/${{ matrix.rust }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@master
+        with:
+          toolchain: ${{ matrix.rust }}
+          components: rustfmt
+      - run: cargo fmt --all --check
+
   tests:
     runs-on: ubuntu-latest
     strategy:
@@ -103,6 +119,7 @@ jobs:
   conclusion:
     needs:
       - clippy
+      # - format  # should format be required?
       - tests
       - cross_test
       - cargo-careful

benches/append.rs

@@ -6,30 +6,27 @@ use test::Bencher;
 use ndarray::prelude::*;
 
 #[bench]
-fn select_axis0(bench: &mut Bencher) {
+fn select_axis0(bench: &mut Bencher)
+{
     let a = Array::<f32, _>::zeros((256, 256));
     let selectable = vec![0, 1, 2, 0, 1, 3, 0, 4, 16, 32, 128, 147, 149, 220, 221, 255, 221, 0, 1];
-    bench.iter(|| {
-        a.select(Axis(0), &selectable)
-    });
+    bench.iter(|| a.select(Axis(0), &selectable));
 }
 
 #[bench]
-fn select_axis1(bench: &mut Bencher) {
+fn select_axis1(bench: &mut Bencher)
+{
     let a = Array::<f32, _>::zeros((256, 256));
     let selectable = vec![0, 1, 2, 0, 1, 3, 0, 4, 16, 32, 128, 147, 149, 220, 221, 255, 221, 0, 1];
-    bench.iter(|| {
-        a.select(Axis(1), &selectable)
-    });
+    bench.iter(|| a.select(Axis(1), &selectable));
 }
 
 #[bench]
-fn select_1d(bench: &mut Bencher) {
+fn select_1d(bench: &mut Bencher)
+{
     let a = Array::<f32, _>::zeros(1024);
     let mut selectable = (0..a.len()).step_by(17).collect::<Vec<_>>();
     selectable.extend(selectable.clone().iter().rev());
 
-    bench.iter(|| {
-        a.select(Axis(0), &selectable)
-    });
+    bench.iter(|| a.select(Axis(0), &selectable));
 }