Merge pull request #9 from LemLib/ci/clang-format-check

👷🏽 Add clang format workflow, fix bug in pr comment workflow
LemLib · Jul 20, 2024 · e5b0c43 · e5b0c43
2 parents 75eb85e + be072f0
commit e5b0c43
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/.github/workflows/clang-format.yml b/.github/workflows/clang-format.yml
@@ -0,0 +1,21 @@
+name: test-clang-format
+
+on:
+  push:
+    branches:
+      - '*'
+  pull_request:
+  workflow_dispatch:
+
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+    - uses: DoozyX/[email protected]
+      with:
+        source: './src/gamepad ./include/gamepad'
+        extensions: 'hpp,cpp'
+        clangFormatVersion: 17
diff --git a/.github/workflows/pr-comment.yml b/.github/workflows/pr-comment.yml
@@ -26,7 +26,7 @@ jobs:
 
               const old_marker = new RegExp(marker.replace("\r\n", "\r?\n")).exec(old_body)?.[0] ?? marker;
               
-              body = old_body.split(old_marker)[0] + marker + body;
+              body = (old_body ?? "").split(old_marker)[0] + marker + body;
               await github.request('PATCH /repos/{owner}/{repo}/pulls/{pull_number}', {
                 owner: owner,
                 repo: repo,