fixing merge report

prayanshchh · Nov 26, 2024 · bec94ad · bec94ad
1 parent 0c23834
commit bec94ad
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -58,8 +58,8 @@ jobs:
       - name: Check for linting errors in modified files
         if: steps.changed-files.outputs.only_changed != 'true'
         env: 
-          CHANGED_FILES: ${{ steps.changed_files.outputs.all_changed_files }}
-        run: npx eslint ${CHANGED_FILES} && python .github/workflows/eslint_disable_check.py
+          CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_files }}
+        run: npx eslint "${CHANGED_FILES}" && python .github/workflows/eslint_disable_check.py
 
       - name: Check for TSDoc comments
         run: npm run check-tsdoc # Run the TSDoc check script
@@ -204,6 +204,7 @@ jobs:
         env:
           NODE_V8_COVERAGE: './.nyc_output'
         run: |
+          mkdir -p ./.nyc_output
           npm run test -- --watchAll=false --coverage
 
       - name: Run Vitest Tests
@@ -216,7 +217,8 @@ jobs:
       - name: Merge Coverage Reports
         if: steps.changed-files.outputs.only_changed != 'true'
         run: |
-          npx nyc merge
+          mkdir -p ./coverage
+          npx nyc merge || exit 1
           mv coverage.json ./coverage/lcov.info     
       
       - name: TypeScript compilation for changed files