ci(astyle): use astyle-check instead of system one

Since Ubuntu 24.4, astyle package does not accept
the same options.

Signed-off-by: Frederic Pillon <[email protected]>

Author: fpistm

.astyleignore

@@ -0,0 +1,9 @@
+.git
+docs
+api-docs
+system
+src/STM32CubeWL
+src/BSP/timer_if.c
+src/BSP/timer_if.h
+src/BSP/rtc.c
+src/BSP/rtc.h

.github/workflows/check-astyle.yml

@@ -1,18 +1,38 @@
 name: Check code formatting with astyle
-
 on:
   push:
+    branches:
+      - main
+    paths-ignore:
+      - '*'
+      - '**.md'
+      - '**.txt'
   pull_request:
+    paths-ignore:
+      - '*'
+      - '**.md'
+      - '**.txt'
   # Allows you to run this workflow manually from the Actions tab
   workflow_dispatch:
 jobs:
-  check:
-    name: Check for astyle errors
+  astyle_check:
     runs-on: ubuntu-latest
-
+    name: Check for astyle errors
     steps:
-      - run: sudo apt install astyle
-      - uses: actions/checkout@v3
-      - run: astyle --project=.astylerc --recursive '*.c' '*.h'
-      # If anything changed, this will fail and show the needed changes
-      - run: git diff --exit-code
+    # First of all, clone the repo using the checkout action.
+    - name: Checkout
+      uses: actions/checkout@main
+
+    - name: Astyle check
+      id: Astyle
+      uses: stm32duino/actions/astyle-check@main
+      with:
+        astyle-definition: '.astylerc'
+        ignore-path-list: '.astyleignore'
+
+    # Use the output from the `Astyle` step
+    - name: Astyle Errors
+      if: failure()
+      run: |
+        cat ${{ steps.Astyle.outputs.astyle-result }}
+        exit 1

.gitignore

@@ -1,5 +1,7 @@
 .clangd
+.development
 .gdb_history
 build
 sketch.json
 api-docs
+astyle.out

src/BSP/mw_log.cpp

@@ -38,10 +38,10 @@
 #include <stdarg.h>
 #include <core_debug.h>
 
-void MW_LOG([[gnu::unused]] MwLogTimestamp_t ts, [[gnu::unused]] MwLogLevel_t level, const char* fmt, ...)
+void MW_LOG([[gnu::unused]] MwLogTimestamp_t ts, [[gnu::unused]] MwLogLevel_t level, const char *fmt, ...)
 {
-    va_list ap;
-    va_start(ap, fmt);
-    vcore_debug(fmt, ap);
-    va_end(ap);
+  va_list ap;
+  va_start(ap, fmt);
+  vcore_debug(fmt, ap);
+  va_end(ap);
 }