Skip to content

feat(ci): add advisory PR review and test coverage checks - #57

Open
kdaula wants to merge 1 commit into
mainfrom
kavi/prt-722-review-bots
Open

kdaula wants to merge 1 commit into
mainfrom
kavi/prt-722-review-bots

Conversation

@kdaula

@kdaula kdaula commented Sep 22, 2026

Copy link
Copy Markdown

Run the PR review and test coverage bots on protos

github-actions[bot]

This comment was marked as outdated.

@bleggett bleggett left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This repo contains nothing but .proto files (specs, no code), which already have strict correctness requirements and are already fully linted by buf for every form of proto mistake - things like compatibility breaks, wire compatibility, naming conventions, refactors, etc.

Given the limited scope of what's in this repo (no code, just specs), the fact that proto files are extremely specified and already fully linted in CI with the buf checkers above, what does adding the Claude review bot get us?

@github-actions

github-actions Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedSep 24, 2026, 4:36 PM

@kdaula
kdaula force-pushed the kavi/prt-722-review-bots branch from b1a3863 to 9af28ba Compare September 24, 2026 16:31
@kdaula
kdaula force-pushed the kavi/prt-722-review-bots branch from 9af28ba to 0936084 Compare September 24, 2026 16:35
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