Skip to content

chore: updates the bazel version adapts branch instructions to our de facto standard. - #399

Merged
zzcgumn merged 3 commits into
developfrom
chore/update_bazel_pin
Sep 23, 2026
Merged

zzcgumn merged 3 commits into
developfrom
chore/update_bazel_pin

Conversation

@zzcgumn

@zzcgumn zzcgumn commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@zzcgumn zzcgumn self-assigned this Sep 21, 2026
@zzcgumn
zzcgumn requested a review from tameware September 21, 2026 21:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Clarify that the / separator used by the prescribed branch names is permitted.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Low severity

Open (1)
What changed in this PR

Updates Bazel to 9.2.0 and standardizes branch naming guidance.

Changes:

  • Pins Bazel to 9.2.0.
  • Regenerates the Bazel module lockfile.
  • Revises branch naming conventions.
File Description
MODULE.bazel.lock Regenerated for Bazel 9.2.0.
.github/​instructions/​github.instructions.md Updates branch naming guidance.
.bazelversion Pins Bazel 9.2.0.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/instructions/github.instructions.md Outdated
Copilot flagged that "no other special characters are allowed" contradicts
the feature/<short_description> style examples immediately above it, which
all use / as a category separator. Clarify that / is the one exception.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved review comments were identified.

Review effort: Lite
Findings: None

Resolved since last review (1)

@zzcgumn zzcgumn added the Clean Copilot review Copilot reviewed and had neither new comments nor new suppressed comments. label Sep 23, 2026
@zzcgumn
zzcgumn merged commit 298db42 into develop Sep 23, 2026
13 checks passed
@zzcgumn
zzcgumn deleted the chore/update_bazel_pin branch September 23, 2026 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Clean Copilot review Copilot reviewed and had neither new comments nor new suppressed comments.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants