Skip to content

Revert "chore(ci): notify coretrace-runner when a release is published" - #293

Merged
cedricroulof merged 1 commit into
mainfrom
revert/notify-runner-on-release
Nov 8, 2026
Merged

cedricroulof merged 1 commit into
mainfrom
revert/notify-runner-on-release

Conversation

@cedricroulof

@cedricroulof cedricroulof commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Reverts #292 (commit 30b2b0f). −16 lines, nothing added. No package code is touched; this removes the notification step from the publish job and restores .github/workflows/release.yml to its pre-#292 state byte for byte.

Why

The equivalent change in coretrace-stack-analyzer (#269) was closed on the grounds that dependency monitoring belongs in the consuming repository rather than in a notification from the producer. That reasoning applies identically here, and the argument is right:

  • this repository should not need to know that it has a consumer, or how that consumer manages its dependencies;
  • the notification required a cross-repository credential for something the consumer can do by itself;
  • any future consumer would otherwise mean another change to this release pipeline.

Leaving #292 merged while #269 is closed would also leave the two release pipelines inconsistent.

What replaces it

Nothing in this repository. Consumers poll this package's releases on their own schedule, which is the usual model for dependency updates.

This reverts commit 30b2b0f, merged as #292.

Dependency monitoring belongs in the consuming repository, not in a dispatch
from the producer: coretrace-runner polls this package's releases on a schedule
instead, so this repository does not need to know it has a consumer and no
cross-repository RUNNER_DISPATCH_TOKEN is required.

The equivalent change in coretrace-stack-analyzer (#269) was closed for the same
reason; this keeps the two release pipelines consistent.
@cedricroulof
cedricroulof merged commit 9b7d0f8 into main Nov 8, 2026
10 checks passed
@SizzleUnrlsd
SizzleUnrlsd deleted the revert/notify-runner-on-release branch November 8, 2026 19:09
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