Repository navigation
ci(deps): the dotnet-tools entries that ship nothing form the repo-tooling group - #336
Conversation
PR SummaryLow Risk Overview
Closes #328 by preventing accidental package releases when only repo build/doc tooling changes. Reviewed by Cursor Bugbot for commit a4bc2ed. Bugbot is set up for automated code reviews on this repo. Configure here. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 05aa6fb5c4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
05aa6fb to
cbc925b
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit cbc925b. Configure here.
…oling group gitversion.tool and defaultdocumentation.console were in no group, so a bump (#283) carried the plain build(deps) subject that .releaserc.json maps to a patch release of nine unchanged packages. They are grouped as repo-tooling, with the security-updates twin test-tooling has, the release rule leaves that name out too, and verify-release-config.mjs checks both subjects. Closes #328 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
cbc925b to
a4bc2ed
Compare

What does this change?
Closes #328.
gitversion.toolanddefaultdocumentation.consolein.config/dotnet-tools.jsonwere in no Dependabot group, so a bump (#283) carried the plainbuild(deps)subject that.releaserc.jsonmaps to a patch release of nine unchanged packages. They are therepo-toolinggroup now (with the security-updates twintest-toolinghas), the release rule's subject glob leaves that name out as it doestest-tooling, andeng/verify-release-config.mjschecks both subjects publish nothing.#330 (grouping the
codeql-actionbumps) turned out to be done onmainalready (3c59765); the commit this branch first carried for it duplicated that group's key and is dropped (Codex and Bugbot on the first head).Type of change
docs/test/refactor/chore/ci— no releaseChecklist
node eng/verify-release-config.mjs 9.9.9passes with the new subjects (repo-tooling,repo-tooling-security→ no release;EdsDcfNetbump → patch)🤖 Generated with Claude Code