ci: split workflow

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

Author: fpistm

.github/workflows/CodeSpell.yml

@@ -0,0 +1,26 @@
+name: codespell
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+  # Allows you to run this workflow manually from the Actions tab
+  workflow_dispatch:
+jobs:
+  codespell:
+    name: Check for spelling errors
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      # See: https://github.com/codespell-project/actions-codespell/blob/master/README.md
+      - name: Spell check
+        uses: codespell-project/actions-codespell@master
+        with:
+          check_filenames: true
+          check_hidden: true
+          # In the event of a false positive, add the word in all lower case to this file:
+          ignore_words_file: ./extras/codespell-ignore-words-list.txt

.github/workflows/Continuous-Integration.yml

@@ -15,39 +15,6 @@ on:
   # Allows you to run this workflow manually from the Actions tab
   workflow_dispatch:
 jobs:
-  astyle_check:
-    runs-on: ubuntu-latest
-    name: AStyle check
-    steps:
-    # 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
-
-    # Use the output from the `Astyle` step
-    - name: Astyle Errors
-      if: failure()
-      run: |
-        cat ${{ steps.Astyle.outputs.astyle-result }}
-        exit 1
-  codespell:
-    name: Check for spelling errors
-    runs-on: ubuntu-latest
-    steps:
-    - name: Checkout
-      uses: actions/checkout@main
-
-    # See: https://github.com/codespell-project/actions-codespell/blob/master/README.md
-    - name: Spell check
-      uses: codespell-project/actions-codespell@master
-      with:
-        check_filenames: true
-        check_hidden: true
-        # In the event of a false positive, add the word in all lower case to this file:
-        ignore_words_file: ./extras/codespell-ignore-words-list.txt
   lib_build:
     runs-on: ubuntu-latest
     name: Library compilation

.github/workflows/check-astyle.yml

@@ -0,0 +1,18 @@
+name: Check code formatting with astyle
+
+on:
+  push:
+  pull_request:
+  # Allows you to run this workflow manually from the Actions tab
+  workflow_dispatch:
+jobs:
+  check:
+    name: Check for astyle errors
+    runs-on: ubuntu-latest
+
+    steps:
+      - run: sudo apt install astyle
+      - uses: actions/checkout@v3
+      - run: astyle --project=extras/.astylerc --recursive '*.c*' '*.h'
+      # If anything changed, this will fail and show the needed changes
+      - run: git diff --exit-code

extras/.astylerc

@@ -0,0 +1,60 @@
+# STM32duino code style definition file for astyle
+
+# Don't create backup files, let git handle it
+suffix=none
+
+# K&R style
+style=kr
+
+# 1 TBS addition to k&r, add braces to one liners
+# Use -j as it was changed in astyle from brackets to braces, this way it is compatible with older astyle versions
+-j
+
+# 2 spaces, convert tabs to spaces
+indent=spaces=2
+convert-tabs
+lineend=linux
+
+# Indent switches and cases
+indent-classes
+indent-switches
+indent-cases
+indent-col1-comments
+indent-preproc-block
+
+# Remove spaces in and around parentheses
+unpad-paren
+
+# Insert a space after if, while, for, and around operators
+pad-header
+pad-oper
+
+# Pointer/reference operators go next to the name (on the right)
+align-pointer=name
+align-reference=name
+
+# Attach { for classes and namespaces
+attach-namespaces
+attach-classes
+
+# Extend longer lines, define maximum 120 value. This results in aligned code,
+# otherwise the lines are broken and not consistent
+max-continuation-indent=120
+
+# if you like one-liners, keep them
+keep-one-line-statements
+
+#remove-comment-prefix
+
+# Do not error our when a directory is excluded that does not contain
+# any files to process
+ignore-exclude-errors
+
+# Exclude some files
+exclude=.git
+exclude=docs
+exclude=api-docs
+
+# Allow oneline blocks, to allow very small functions to be defined
+# on a single line in header files.
+keep-one-line-blocks