Skip to content

v1.9.1#133

Merged
BakerNet merged 1 commit into
mainfrom
post/v1.9.1
May 20, 2026
Merged

v1.9.1#133
BakerNet merged 1 commit into
mainfrom
post/v1.9.1

Conversation

@BakerNet
Copy link
Copy Markdown
Collaborator

Post v1.9.1

@github-actions github-actions Bot requested a review from zbedforrest May 20, 2026 18:51
@github-actions
Copy link
Copy Markdown

Codeowners approval required for this PR:

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the codeowners-plus action version in the README to v1.9.1 and increments the CLI version in tools/cli/main.go. A review comment points out a version mismatch, suggesting that the CLI version should be set to v1.9.1 instead of v1.9.2.dev to align with the release documentation and ensure the compiled binary reports the correct version.

Comment thread tools/cli/main.go
Name: "codeowners-cli",
Usage: "CLI tool for working with .codeowners files",
Version: "v1.9.1.dev",
Version: "v1.9.2.dev",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

The version is being bumped to v1.9.2.dev, but the pull request title and README updates indicate this is the release for v1.9.1. If this is the release PR, the version in the code should be set to v1.9.1. Skipping the release version in the code means the compiled binary for v1.9.1 will incorrectly report itself as a development build for v1.9.2.

Suggested change
Version: "v1.9.2.dev",
Version: "v1.9.1",

@BakerNet BakerNet merged commit 21b948d into main May 20, 2026
8 of 10 checks passed
@BakerNet BakerNet deleted the post/v1.9.1 branch May 20, 2026 18:55
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