Merge pull request #422 from Workiva/setup-gha-dart-oss

FED-4060 Set up gha-dart-oss

Author: btr-rmconsole-2[bot]

.github/workflows/dart_ci.yml renamed to .github/workflows/ci.yml

@@ -12,30 +12,45 @@ on:
 jobs:
   # Run as a separate job outside the Dart SDK matrix below,
   # since we can only emit a single SBOM.
-  create-sbom-release-asset:
-    name: Create SBOM Release Asset
+  build:
+    uses: Workiva/gha-dart-oss/.github/workflows/[email protected]
+
+  checks:
+    uses: Workiva/gha-dart-oss/.github/workflows/[email protected]
+    with:
+      additional-checks: |
+        no_entrypoint_imports
+
+  # Run Dart 3 analysis as a separate job outside checks above in order to pre-run the
+  # Dart-2-only file removal script. Consolidate this job with checks above once checks.yaml
+  # supports a pre-run option or we upgrade to Dart 3 and remove the files.
+  stable-analyze:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
       - uses: dart-lang/setup-dart@v1
         with:
-          sdk: 2.19.6 # This version doesn't matter so long as it resolves.
+          sdk: stable
+
+      - name: Print Dart SDK version
+        run: dart --version
       - run: dart pub get
-      - name: Publish SBOM to Release Assets
-        uses: anchore/sbom-action@v0
-        with:
-          path: ./
-          format: cyclonedx-json
 
-  build:
+      - name: Delete Dart-2-only files when running on Dart 3
+        run: ./tool/delete_dart_2_only_files.sh
+
+      - name: Analyze project source
+        run: dart analyze
+
+  test:
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
       matrix:
         react: [17, 18]
         sdk: [2.19.6, stable]
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - id: setup-dart
         uses: dart-lang/setup-dart@v1
         with:
@@ -50,6 +65,7 @@ jobs:
           if [[ "$DART_VERSION" =~ ^3 ]]; then
             ./tool/delete_dart_2_only_files.sh
           fi
+
       - name: Switch to React 17 Test HTML
         if: ${{ matrix.react == 17 }}
         run: |
@@ -60,18 +76,6 @@ jobs:
         name: Install dependencies
         run: dart pub get
 
-      - name: Validate dependencies
-        run: dart run dependency_validator
-        if: always() && steps.install.outcome == 'success'
-
-      - name: Verify formatting
-        run: dart run dart_dev format --check
-        if: ${{ matrix.sdk == '2.19.6' }}
-
-      - name: Analyze project source
-        run: dart analyze
-        if: always() && steps.install.outcome == 'success'
-
       - name: Run tests (DDC)
         run: |
           dart run build_runner test --delete-conflicting-outputs -- --preset dartdevc --preset=react${{ matrix.react }}
@@ -83,3 +87,16 @@ jobs:
           dart run build_runner test --delete-conflicting-outputs --release -- --preset dart2js --preset=react${{ matrix.react }}
         if: always() && steps.install.outcome == 'success'
         timeout-minutes: 8
+
+  testing-and-checks-complete:
+    name: Testing and Checks Completed
+    needs: [ build, checks, test ]
+    if: ${{ !cancelled() && github.event_name == 'pull_request' || github.ref_type == 'tag' }}
+    runs-on: ubuntu-latest
+    steps:
+      - run: |
+          if [ "${{ needs.checks.result }}" == "failure" ]; then
+            echo "Some checks have failed. Please fix the issues and try again."
+            exit 1
+          fi
+          echo 'All Testing and checks have completed.'

.github/workflows/publish.yml

@@ -0,0 +1,15 @@
+name: Publish
+
+on:
+  push:
+    tags:
+      - '[0-9]+.[0-9]+.[0-9]+'
+
+permissions:
+  contents: write
+  id-token: write
+  pull-requests: write
+
+jobs:
+  publish:
+    uses: Workiva/gha-dart-oss/.github/workflows/[email protected]

CHANGELOG.md

@@ -1,3 +1,7 @@
+## Unreleased
+- Set up gha-dart-oss
+- Fix no_entrypoint_imports warnings
+
 ## 7.3.0
 - Add new, opt-in React 18 JS files (see [README](https://github.com/Workiva/react-dart#html) for more info)
    - The preexisting JS files that use React 17 are now deprecated, and will be removed in the next major version, 8.0.0.

pubspec.yaml

@@ -12,7 +12,7 @@ dev_dependencies:
   build_runner: ^2.1.2
   build_test: ^2.1.3
   build_web_compilers: '>=3.0.0 <5.0.0'
-  dart_dev: ^4.0.0
+  dart_dev: ^4.2.4
   dependency_validator: ^3.2.2
   glob: ^2.0.0
   matcher: ^0.12.11

tool/dart_dev/config.dart

@@ -22,4 +22,5 @@ final config = {
       // Don't format generated mockito files.
       Glob('test/mockito.mocks.dart'),
     ],
+  'analyze': AnalyzeTool(),
 };