Skip to content

Run analyze, format, and tests on every push and pull request - #12

Merged
justintout merged 1 commit into
mainfrom
feature/ci
Sep 15, 2026
Merged

justintout merged 1 commit into
mainfrom
feature/ci

Conversation

@justintout

Copy link
Copy Markdown
Owner

Adds .github/workflows/test.yml, which runs on every push to main and on every pull request.

Steps, matching what the package must already pass:

  • dart pub get
  • dart analyze --fatal-warnings
  • dart format --output=none --set-exit-if-changed .
  • dart test

All pass on this branch locally on Dart 3.10.4: analyze reports no issues, format reports 0 of 7 files changed, and 26 of 26 tests pass.

This closes the CI request in #8's thread. That issue is about a test that was failing silently; a workflow like this is what would have caught it.

Issue #8's thread asked for CI on pull requests. The repo had no workflow,
so regressions like the failing "the bug" test could sit unnoticed.
@justintout
justintout merged commit 728f696 into main Sep 15, 2026
1 check passed
@justintout
justintout deleted the feature/ci branch September 15, 2026 03:46
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