Skip to content

ci: Add a second GitHub actions workflow#5

Merged
johnpryan merged 1 commit intoflutter:mainfrom
jwren:second-github-workflow-analyzer-2
Mar 2, 2026
Merged

ci: Add a second GitHub actions workflow#5
johnpryan merged 1 commit intoflutter:mainfrom
jwren:second-github-workflow-analyzer-2

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 - is currently omitted as the test suite needs to be refactored to work properly without relying on real API keys in the CI environment

#23

@jwren jwren force-pushed the second-github-workflow-analyzer-2 branch from 6585cdb to 7162e52 Compare March 2, 2026 18:22
@jwren jwren changed the title ci: add skills tool verification workflow ci: Add a second GitHub actions workflow Mar 2, 2026
@jwren jwren force-pushed the second-github-workflow-analyzer-2 branch 2 times, most recently from e2ccbdf to 76ae109 Compare March 2, 2026 18:32
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` - is currently omitted as the test suite needs to be refactored to work properly without relying on real API keys in the CI environment

flutter/flutter#183124
@jwren jwren force-pushed the second-github-workflow-analyzer-2 branch from 76ae109 to 11b0667 Compare March 2, 2026 18:41
@jwren
Copy link
Member Author

jwren commented Mar 2, 2026

@johnpryan

- run: dart analyze --fatal-infos

# NOTE: dart test is currently omitted as the test suite needs to be
# refactored to work properly without relying on real API keys in the CI environment
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do the tests use Gemini? That's probably not what we want.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #8 fake keys need to be set for tests to pass.

@johnpryan johnpryan merged commit 88c4b24 into flutter:main Mar 2, 2026
6 checks passed
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.

2 participants