Skip to content

ci: Remove --order-dependents from release publish trigger - #4084

Merged
spydon merged 1 commit into
mainfrom
ci/remove-order-dependents
Oct 1, 2026
Merged

spydon merged 1 commit into
mainfrom
ci/remove-order-dependents

Conversation

@spydon

@spydon spydon commented Oct 1, 2026

Copy link
Copy Markdown
Member

Description

The Tag release workflow failed to dispatch any publish runs for the last two releases (#4077 and #4010) because melos aborts on the dependency cycle between `flame` and `flame_test` (`flame_test` is a dev dependency of `flame`) when `--order-dependents` is used:

```
🚨 1 cycles in dependencies found:
[ flame -> flame_test ]
Failed to update packages.
```

The ordering also gave no real guarantee, since `gh workflow run` returns immediately and the dispatched publish runs execute in parallel regardless of dispatch order. This removes the flag so the dispatch step works again.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

@spydon
spydon enabled auto-merge (squash) October 1, 2026 12:41
@spydon
spydon merged commit 53053ea into main Oct 1, 2026
9 checks passed
@spydon
spydon deleted the ci/remove-order-dependents branch October 1, 2026 12:47
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