| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
No active review issues block the migration of bundle-update release notes to the changenote pipeline.
No open issues 🎉
What changed in this PRMoves bundle-update release notes into the existing changenote pipeline, so bundle update PRs no longer modify CHANGELOG.md directly or need an exception to the check proposed in #4217.
Changes:
| File | Description |
|---|---|
| pr-checks/bundle-changelog.ts | Writes a bundle-update changenote instead of editing the changelog. |
| pr-checks/bundle-changelog.test.ts | Tests generated note contents and compatibility with changenote validation. |
| .github/workflows/update-bundle.yml | Stages and commits the new changenote files. |
Review effort: Balanced (auto)
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
The "Update default CodeQL bundle" workflow currently adds its "Update default CodeQL bundle version" entry directly to the [UNRELEASED] section of CHANGELOG.md. This PR changes pr-checks/bundle-changelog.ts to write the entry to a changenote file in unreleased-change-notes/ instead, so that it's added to the changelog with the other changenotes when we release.
The workflow now stages the unreleased-change-notes directory, since git commit -a doesn't pick up new files.
With this change, bundle update PRs no longer modify CHANGELOG.md, so they won't need to be excluded from the check added in #4217.
Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
Workflow types:
Products:
Environments:
How did/will you validate this change?
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
How will you know if something goes wrong after this change is released?
Are there any special considerations for merging or releasing this change?
Merge / deployment checklist