Merge pull request #4514 from smoogipoo/fix-resharper-ci

Make Resharper inspections fail the CI job
ppy · Jun 16, 2021 · 3c860da · 3c860da
2 parents 6ca6f06 + 88def79
commit 3c860da
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.config/dotnet-tools.json b/.config/dotnet-tools.json
@@ -20,8 +20,8 @@
         "jb"
       ]
     },
-    "nvika": {
-      "version": "2.0.0",
+    "smoogipoo.nvika": {
+      "version": "1.0.1",
       "commands": [
         "nvika"
       ]

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -91,7 +91,5 @@ jobs:
       - name: InspectCode
         run: dotnet jb inspectcode $(pwd)/osu-framework.Desktop.slnf --output=$(pwd)/inspectcodereport.xml --cachesDir=$(pwd)/inspectcode --verbosity=WARN
 
-      - name: ReSharper
-        uses: glassechidna/resharper-action@master
-        with:
-          report: ${{github.workspace}}/inspectcodereport.xml
+      - name: NVika
+        run: dotnet nvika parsereport "${{github.workspace}}/inspectcodereport.xml" --treatwarningsaserrors