Small ci conf update (#164)

* Update `clang-format` conf * Update `yamllint` * Fix clang-16 * Oopsie * More oopsie
EnAccess · Feb 26, 2024 · 187a67e · 187a67e
1 parent 559a9ff
commit 187a67e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/.yamllint b/.yamllint
@@ -9,5 +9,6 @@ rules:
     quote-type: double
     required: only-when-needed
     allow-quoted-quotes: true
+    check-keys: true
   truthy:
     check-keys: false
diff --git a/firmware/.clang-format b/firmware/.clang-format
@@ -6,6 +6,11 @@ AllowShortFunctionsOnASingleLine: Empty
 IncludeBlocks: Preserve
 IndentPPDirectives: AfterHash
 DerivePointerAlignment: false
+# This is only available in Clang-16, see
+# https://clang.llvm.org/docs/ClangFormatStyleOptions.html#insertnewlineateof
+# However, Github runners currently only ship with Clang-15, see
+# https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md
+# InsertNewlineAtEOF: true
 
 # Always break after if to get accurate coverage
 AllowShortIfStatementsOnASingleLine: false

diff --git a/firmware_selftest/.clang-format b/firmware_selftest/.clang-format
@@ -6,6 +6,11 @@ AllowShortFunctionsOnASingleLine: Empty
 IncludeBlocks: Preserve
 IndentPPDirectives: AfterHash
 DerivePointerAlignment: false
+# This is only available in Clang-16, see
+# https://clang.llvm.org/docs/ClangFormatStyleOptions.html#insertnewlineateof
+# However, Github runners currently only ship with Clang-15, see
+# https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md
+# InsertNewlineAtEOF: true
 
 # Always break after if to get accurate coverage
 AllowShortIfStatementsOnASingleLine: false