update the specification validation workflow (#3351)

dart-lang · Sep 15, 2023 · b19bf29 · b19bf29
1 parent 4d1e996
commit b19bf29
Showing 1 changed file with 9 additions and 12 deletions.
diff --git a/.github/workflows/ci.yml → .github/workflows/spec.yml b/.github/workflows/ci.yml → .github/workflows/spec.yml
@@ -1,32 +1,25 @@
-name: CI
+name: Spec
 
 on:
   # Run on PRs and pushes to the default branch.
   push:
     branches: [ main ]
     paths:
       - 'specification/**'
-      - '.github/workflows/ci.yml'
+      - '.github/workflows/spec.yml'
   pull_request:
     branches: [ main ]
     paths:
       - 'specification/**'
-      - '.github/workflows/ci.yml'
+      - '.github/workflows/spec.yml'
 
 jobs:
   specification:
     runs-on: ubuntu-latest
-    if: |
-      (
-        (github.event_name == 'push'
-          && github.ref == 'refs/heads/main'
-          && github.repository == 'dart-lang/language'
-        ) ||
-        (github.event.pull_request.head.repo.full_name == github.repository)
-      )
 
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9
+
       - name: Install latex tools
         run: |
           sudo apt-get update -qq
@@ -35,13 +28,17 @@ jobs:
             texlive-latex-extra \
             texlive-fonts-recommended \
             lmodern
+
       - name: Build specification
         run: |
           cd specification
           make
           mkdir firebase
           cp dartLangSpec.pdf firebase/DartLangSpecDraft.pdf
-      - uses: FirebaseExtended/action-hosting-deploy@120e124148ab7016bec2374e5050f15051255ba2
+
+      - name: Upload specification
+        if: ${{ github.event.pull_request.merged }}
+        uses: FirebaseExtended/action-hosting-deploy@120e124148ab7016bec2374e5050f15051255ba2
         with:
           repoToken: '${{ secrets.GITHUB_TOKEN }}'
           firebaseServiceAccount: '${{ secrets.FIREBASE_SERVICE_ACCOUNT_DART_SPECIFICATION }}'