Skip to content

FED-4060 Set up gha-dart-oss #422

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

sydneyjodon-wk
Copy link
Collaborator

@sydneyjodon-wk sydneyjodon-wk commented Aug 7, 2025

We should set up gha-dart-oss so we can:

  • take advantage of shared workflows
  • use the publish workflow to auto-publish on merge 🎉

QA instructions

  • CI passes
  • All steps previously run in CI are still running even if some of them are in the shared jobs now
  • When this merges, follow along to make sure the package actually publishes

- name: Analyze project source
run: dart analyze

test:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Because I changed this from "build" to "test" - I'll update Rosie to check for the new-named CI checks before this merges

- name: Analyze project source
run: dart analyze

test:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We can't use the shared unit-test from gha-dart-oss yet because we can't run other commands beforehand - see slack thread

@sydneyjodon-wk sydneyjodon-wk marked this pull request as ready for review August 8, 2025 23:09
@btr-rmconsole-3 btr-rmconsole-3 bot changed the title Set up gha-dart-oss FED-4060 Set up gha-dart-oss Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant