RFC: Try out running pre-made pre-commit hooks via github actions

Author: paulbartell

.github/scripts/kernel_checker.py

@@ -12,6 +12,7 @@
 
 KERNEL_IGNORED_EXTENSIONS = [
     '.yml',
+    '.yaml',
     '.css',
     '.idx',
     '.md',
@@ -92,4 +93,3 @@ def main():
 
 if __name__ == '__main__':
     exit(main())
-

.github/workflows/ci.yml

@@ -1,7 +1,5 @@
 name: CI Checks
 on:
-  push:
-    branches: ["**"]
   pull_request:
     branches: [main]
   workflow_dispatch:
@@ -44,4 +42,3 @@ jobs:
       - name: URL Checker
         run: |
             bash kernel/.github/actions/url_verifier.sh kernel
-

.github/workflows/pre-commit.yml

@@ -0,0 +1,18 @@
+name: Linters
+
+on:
+  pull_request:
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/setup-python@v2
+    - id: file_changes
+      uses: trilom/[email protected]
+      with:
+        output: ' '
+    - uses: pre-commit/[email protected]
+      with:
+        extra_args: --files ${{ steps.file_changes.outputs.files}}

.pre-commit-config.yaml

@@ -0,0 +1,18 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v3.4.0
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-yaml
+    -   id: check-added-large-files
+    -   id: mixed-line-ending
+        args : [--fix=auto]
+    -   id: check-case-conflict
+    -   id: check-executables-have-shebangs
+    -   id: check-json
+    -   id: check-merge-conflict
+    -   id: check-symlinks
+    -   id: check-ast

event_groups.c

@@ -26,6 +26,7 @@
 
 /* Standard includes. */
 #include <stdlib.h>
+/* make a change that breaks things */
 
 /* Defining MPU_WRAPPERS_INCLUDED_FROM_API_FILE prevents task.h from redefining
  * all the API functions to use the MPU wrappers.  That should only be done when