ci: enforce static analysis (#227)

bdlukaa · Mar 15, 2022 · b99d1f5 · b99d1f5
2 parents 35e2584 + f959a9a
commit b99d1f5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -4,9 +4,6 @@
 
 <!-- Mark all that applyes -->
 
-- [ ] I have run `dartfmt` on all changed files <!-- REQUIRED --> 
 - [ ] I have updated `CHANGELOG.md` with my changes <!-- REQUIRED --> 
 - [ ] I have run "optimize/organize imports" on all changed files
-- [ ] I have addressed all analyzer warnings as best I could
-- [ ] I have added/updated relevant documentation
-- [ ] I have run `flutter pub publish --dry-run` and addressed any warnings
+- [ ] I have added/updated relevant documentation
diff --git a/.github/workflows/flutter_analysis.yml b/.github/workflows/flutter_analysis.yml
@@ -17,8 +17,17 @@ jobs:
         with:
           channel: stable
 
-      - name: install dependencies
+      - name: Install dependencies
         run: flutter pub get
 
+      - name: Check formatting
+        run: flutter format --set-exit-if-changed .
+
+      - name: Run analysis
+        run: flutter analyze
+
+      - name: Verify package health
+        run: flutter pub publish --dry-run
+
       - name: Run tests
         run: flutter test