Merge pull request #2297 from AllenInstitute/feature/2297-add-linting

CI: Rework ipt format check
AllenInstitute · Nov 20, 2024 · 5fcb28b · 5fcb28b
2 parents a4d0676 + 501bbda
commit 5fcb28b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/build-pr.yml b/.github/workflows/build-pr.yml
@@ -39,7 +39,7 @@ concurrency:
 jobs:
 
   Ipt:
-    name: 🔎 IPT Format check
+    name: 🔎 IPT checks
     runs-on: [ self-hosted, Windows ]
     steps:
       - name: Checkout repository
@@ -49,8 +49,8 @@ jobs:
         run: curl -o ipt.exe "https://byte-physics.de/public-downloads/aistorage/transfer/ipt/0.6.0/ipt.exe"
       - name: IPT version
         run: ./ipt.exe --version
-      - name: Format code
-        run: tools/ipt-format.sh
+      - name: Format and lint code
+        run: tools/run-ipt.sh
       - name: Check for changed files
         run: git diff --name-only --ignore-submodules; git diff  --ignore-submodules --quiet
       - name: Create patch

diff --git a/tools/ipt-format.sh → tools/run-ipt.sh b/tools/ipt-format.sh → tools/run-ipt.sh
@@ -23,9 +23,8 @@ else
   ipt="ipt"
 fi
 
-echo "[format]" > config.toml
+echo "[lint]" > config.toml
 while read -r line; do
     echo "files = \"$line\"" >> config.toml
 done < <(git ls-files ':(attr:ipt)')
-
-(cd $top_level && $ipt --arg-file config.toml format -i)
+(cd $top_level && $ipt --arg-file config.toml lint -i)