Skip to content

ci: use very_good_workflows/dart_pub_publish - #85

Merged
marcossevilla merged 2 commits into
mainfrom
vgv-ai-bot/issue-84
Aug 10, 2026
Merged

ci: use very_good_workflows/dart_pub_publish#85
marcossevilla merged 2 commits into
mainfrom
vgv-ai-bot/issue-84

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Warning

The coding agent reported this implementation is incomplete.
The workflow now calls dart_pub_publish, but it requires a PUB_CREDENTIALS repo secret to be configured since the reusable workflow uses credential-based auth instead of the previous OIDC setup.

This PR has been opened as a draft. Review the diff against the original issue before marking it ready for review.

Summary

Replaces the hand-rolled pub publish job with the reusable dart_pub_publish workflow from very_good_workflows.

Changes

  • Removed the manual checkout / Dart & Flutter setup / dry-run / publish steps in .github/workflows/pub_publish.yaml.
  • Now calls VeryGoodOpenSource/very_good_workflows/.github/workflows/dart_pub_publish.yml@v1.
  • Pins dart_sdk to 3.12.0 and passes PUB_CREDENTIALS via the pub_credentials secret.

Why

Using the shared workflow reduces duplication and keeps publishing aligned with VGV's maintained CI standards.

Closes #84

Closes #84

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
Comment thread .github/workflows/pub_publish.yaml Outdated
@marcossevilla
marcossevilla marked this pull request as ready for review August 10, 2026 12:42
@marcossevilla
marcossevilla requested a review from a team as a code owner August 10, 2026 12:42
@marcossevilla
marcossevilla merged commit ba44df9 into main Aug 10, 2026
4 checks passed
@marcossevilla
marcossevilla deleted the vgv-ai-bot/issue-84 branch August 10, 2026 12:43
@github-project-automation github-project-automation Bot moved this from Needs Triage to Done in VGV Open Source 🦄 🧙🌟 Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

ci: use very_good_workflows/dart_pub_publish

1 participant