Add CI job to run cargo fmt.

nrf-rs · Apr 8, 2024 · f913ec5 · f913ec5
1 parent 1049911
commit f913ec5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -44,10 +44,20 @@ jobs:
           - rust_version: stable
             rustflags: --deny warnings
 
+  format:
+    name: Format
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Copy Cargo.toml
+        run: cp Cargo.ci.toml Cargo.toml
+      - name: Format Rust code
+        run: cargo fmt --all -- --check
+
   ci:
     if: ${{ success() }}
     # all new jobs must be added to this list
-    needs: [build]
+    needs: [build, format]
     runs-on: ubuntu-latest
     steps:
       - name: CI succeeded

diff --git a/nrf-hal-common/src/i2s.rs b/nrf-hal-common/src/i2s.rs
@@ -682,10 +682,7 @@ struct Inner<B> {
 impl<B> Transfer<B> {
     /// Returns `true` if the transfer is done.
     pub fn is_done(&self) -> bool {
-        if let Some(inner) = self
-            .inner
-            .as_ref()
-        {
+        if let Some(inner) = self.inner.as_ref() {
             inner.i2s.is_event_triggered(I2SEvent::RxPtrUpdated)
                 || inner.i2s.is_event_triggered(I2SEvent::TxPtrUpdated)
         } else {