Merge pull request #36 from artichoke/lopopolo/clang-format

Lock clang-format dep and lint C code in CI

.clang-format-ignore

@@ -0,0 +1,6 @@
+.git
+build
+emsdk
+node_modules
+target
+vendor

.github/workflows/ci.yaml

@@ -159,6 +159,24 @@ jobs:
       - name: Lint and check formatting with Rubocop
         run: bundle exec rubocop --format github
 
+  c:
+    name: Lint and format C
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4.1.1
+
+      - name: Setup Node.js runtime
+        uses: actions/setup-node@v4.0.1
+        with:
+          node-version: "lts/*"
+
+      - name: Install toolchain
+        run: npm ci
+
+      - name: Lint and check formatting with clang-format
+        run: npm run fmt:c -- --check
+
   text:
     name: Lint and format text
     runs-on: ubuntu-latest

Rakefile

@@ -48,12 +48,12 @@ namespace :format do
 
   desc 'Format text, YAML, and Markdown sources with prettier'
   task :text do
-    sh 'npx prettier --write "**/*"'
+    sh 'npm run fmt'
   end
 
   desc 'Format .c and .h sources with clang-format'
   task :c do
-    sh 'npx --yes github:artichoke/clang-format'
+    sh 'npm run fmt:c'
   end
 end
 
@@ -68,12 +68,12 @@ namespace :fmt do
 
   desc 'Format text, YAML, and Markdown sources with prettier'
   task :text do
-    sh 'npx prettier --write "**/*"'
+    sh 'npm run fmt'
   end
 
   desc 'Format .c and .h sources with clang-format'
   task :c do
-    sh 'npx --yes github:artichoke/clang-format'
+    sh 'npm run fmt:c'
   end
 end
 

package.json

@@ -14,9 +14,11 @@
   "homepage": "https://www.artichokeruby.org",
   "bugs": "https://github.com/artichoke/sysdir-rs/issues",
   "devDependencies": {
+    "@artichokeruby/clang-format": "github:artichoke/clang-format#semver:^0.16.0",
     "prettier": "^3.2.5"
   },
   "scripts": {
-    "fmt": "prettier --write \"**/*\""
+    "fmt": "prettier --write \"**/*\"",
+    "fmt:c": "clang-format"
   }
 }