simplify the workflow for package:graphs

dart-lang · Nov 7, 2024 · 6eeaba9 · 6eeaba9
1 parent d6a2d0c
commit 6eeaba9
Showing 1 changed file with 15 additions and 38 deletions.
diff --git a/.github/workflows/graphs.yml b/.github/workflows/graphs.yml
@@ -1,8 +1,9 @@
 name: package:graphs
+
 permissions: read-all
 
 on:
-  # Run on PRs and pushes to the default branch.
+  # Run CI on pushes to the main branch, and on PRs against main.
   push:
     branches: [ main ]
     paths:
@@ -14,54 +15,30 @@ on:
       - '.github/workflows/graphs.yml'
       - 'pkgs/graphs/**'
   schedule:
-    - cron: "0 0 * * 0"
+    - cron: '0 0 * * 0' # weekly
 
-env:
-  PUB_ENVIRONMENT: bot.github
+defaults:
+  run:
+    working-directory: pkgs/graphs
 
 jobs:
-  # Check code formatting and static analysis on a single OS (linux)
-  # against Dart beta.
-  analyze:
+  build:
     runs-on: ubuntu-latest
-    defaults:
-      run:
-        working-directory: pkgs/graphs/
     strategy:
       fail-fast: false
       matrix:
-        sdk: [dev]
+        sdk: [stable, dev]
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
       - uses: dart-lang/setup-dart@0a8a0fc875eb934c15d08629302413c671d3f672
         with:
           sdk: ${{ matrix.sdk }}
-      - id: install
-        run: dart pub get
-      - run: dart format --output=none --set-exit-if-changed .
-        if: always() && steps.install.outcome == 'success'
+
+      - run: dart pub get
+
       - run: dart analyze --fatal-infos
-        if: always() && steps.install.outcome == 'success'
 
-  test:
-    needs: analyze
-    runs-on: ${{ matrix.os }}
-    defaults:
-      run:
-        working-directory: pkgs/graphs/
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest]
-        sdk: [3.4, dev]
-    steps:
-      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
-      - uses: dart-lang/setup-dart@0a8a0fc875eb934c15d08629302413c671d3f672
-        with:
-          sdk: ${{ matrix.sdk }}
-      - id: install
-        run: dart pub get
-      - run: dart test --platform vm
-        if: always() && steps.install.outcome == 'success'
-      - run: dart test --platform chrome
-        if: always() && steps.install.outcome == 'success'
+      - run: dart format --output=none --set-exit-if-changed .
+        if: ${{ matrix.sdk == 'stable' }}
+
+      - run: dart test