Skip to content

Add a second GitHub actions workflow#4

Closed
jwren wants to merge 6 commits intoflutter:mainfrom
jwren:second-github-workflow-analyzer
Closed

Add a second GitHub actions workflow#4
jwren wants to merge 6 commits intoflutter:mainfrom
jwren:second-github-workflow-analyzer

Conversation

@jwren
Copy link
Member

@jwren jwren commented Mar 2, 2026

Adds a GitHub Actions workflow to automatically run CI checks for the skills CLI tool. Tests are run on both Ubuntu, Mac and Windows for the stable and dev Dart SDK channels. The workflow triggers on pushes and pull requests to the main branch that modify the tool/ directory, and runs:

  • dart pub get
  • dart analyze --fatal-infos
  • dart format (on the dev channel only)
  • dart test

Adds a GitHub Actions workflow to automatically run CI checks for the skills CLI tool. Tests are run on both Ubuntu, Mac and Windows for the `stable` and `dev` Dart SDK channels.
The workflow triggers on pushes and pull requests to the main branch that modify the `tool/` directory, and runs:
- `dart pub get`
- `dart analyze --fatal-infos`
- `dart format` (on the dev channel only)
- `dart test`
@jwren jwren force-pushed the second-github-workflow-analyzer branch from 7b3a3a2 to c575b2b Compare March 2, 2026 07:45
@jwren jwren force-pushed the second-github-workflow-analyzer branch from 3e9e430 to 35bf4ef Compare March 2, 2026 15:19
jwren added 4 commits March 2, 2026 07:24
Skips the tool/resources directory exists check since these assertions need to be moved to the generation script instead.
Skips the skills directory existence check in test/skill_assertions_test.dart since these assertions need to be moved to the generation script instead.
Updates the skip reason on `skills directory exists` and `tool/resources directory exists` to clarify that not all skills will be read/written to these default project directories.
@jwren
Copy link
Member Author

jwren commented Mar 2, 2026

Closing this PR, work is going to start over now that #3 has landed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant