Skip to content

Initially run update-changelogs only for non-draft PRs#8750

Merged
mcmire merged 2 commits intomainfrom
change-update-changelogs-event
May 8, 2026
Merged

Initially run update-changelogs only for non-draft PRs#8750
mcmire merged 2 commits intomainfrom
change-update-changelogs-event

Conversation

@mcmire
Copy link
Copy Markdown
Contributor

@mcmire mcmire commented May 8, 2026

Explanation

When an engineer opens a release PR, they may want to make additional changes to the release plan before they are ready to open the PR, and so they may not want to populate dependency bump entries across changelogs right away. To account for this, this commit modifies the update-changelogs workflow so that the initial run only occurs if the PR is created in non-draft status or is taken out of draft otherwise.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Low risk workflow gating change; it only adjusts when the GitHub Actions job runs and does not affect production code paths.

Overview
Updates the update-changelogs GitHub Actions workflow to not run automatically for draft PRs.

The workflow now triggers on pull_request_target ready_for_review and tightens the is-fork job condition so the initial run only occurs when the PR is not a draft (while still allowing manual runs via @metamaskbot update-changelogs comments).

Reviewed by Cursor Bugbot for commit f8a1fc0. Bugbot is set up for automated code reviews on this repo. Configure here.

When an engineer opens a release PR, they may want to make additional
changes to the release plan before they are ready to open the PR, and
they may not want to populates dependency bump entries across changelogs
right away. To account for this, this commit modifies the
`update-changelogs` workflow so that the initial run only occurs if the
PR is created in non-draft status or is taken out of draft otherwise.
@mcmire mcmire temporarily deployed to default-branch May 8, 2026 14:56 — with GitHub Actions Inactive
@mcmire mcmire marked this pull request as ready for review May 8, 2026 15:02
@mcmire mcmire requested a review from a team as a code owner May 8, 2026 15:02
Comment thread .github/workflows/update-changelogs.yml Outdated
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmire mcmire added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit 0c7feb0 May 8, 2026
366 checks passed
@mcmire mcmire deleted the change-update-changelogs-event branch May 8, 2026 17:27
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