Merge pull request #5876 from ampproject/skip-eslint-report-on-pr-fork

Run ESLint on PR fork without generating report
ampproject · Feb 15, 2021 · 6224acd · 6224acd
2 parents 17a6a71 + 824b36d
commit 6224acd
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/.github/workflows/build-test-measure.yml b/.github/workflows/build-test-measure.yml
@@ -68,7 +68,13 @@ jobs:
       - name: Validate package.json
         run: npm run lint:pkg-json
 
-      - name: Detect coding standard violations (ESLint)
+      - name: Detect ESLint coding standard violations
+        if: github.event.pull_request.head.repo.fork == true
+        run: npm run lint:js
+
+      - name: Generate ESLint coding standard violations report
+        # Prevent generating the ESLint report if run from a PR fork
+        if: github.event.pull_request.head.repo.fork == false
         run: npm run lint:js:report
         continue-on-error: true