Skip to content

Commit

Permalink
Trigger CI on pubspec.* changes (#230)
Browse files Browse the repository at this point in the history
## Description

Currently, our CI pipeline only runs when `*.dart` files are modified.
This PR extends the CI trigger to include changes to `pubspec.*` files,
ensuring that dependency updates are also properly tested.

## Summary (check all that apply)

- [ ] Modified / added code
- [ ] Modified / added tests
- [ ] Modified / added examples
- [x] Modified / added others (pubspec.yaml, workflows, etc...)
- [ ] Updated README
- [ ] Contains breaking changes
  - [ ] Created / updated migration guide
- [ ] Incremented version number
  - [ ] Updated CHANGELOG
  • Loading branch information
fujidaiti authored Aug 24, 2024
1 parent 14e3987 commit 276f855
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/code_check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,20 +15,22 @@ jobs:
permissions:
pull-requests: read
outputs:
dart-files: ${{ steps.filter.outputs.dart-files }}
flutter-files: ${{ steps.filter.outputs.flutter-files }}
steps:
- uses: actions/checkout@v4
- uses: dorny/paths-filter@v3
id: filter
with:
filters: |
dart-files:
flutter-files:
- '**.dart'
- 'pubspec.yaml'
- 'pubspec.lock'
# Static code analysis
analysis:
needs: changes
if: ${{ needs.changes.outputs.dart-files == 'true' }}
if: ${{ needs.changes.outputs.flutter-files == 'true' }}
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand All @@ -49,7 +51,7 @@ jobs:
# Unit testing
testing:
needs: changes
if: ${{ needs.changes.outputs.dart-files == 'true' }}
if: ${{ needs.changes.outputs.flutter-files == 'true' }}
runs-on: ubuntu-latest
permissions:
contents: read
Expand Down

0 comments on commit 276f855

Please sign in to comment.