feat: Move baz reviewer to a separate category from other reviewers#99
Merged
feat: Move baz reviewer to a separate category from other reviewers#99
Conversation
![baz-reviewer[bot]](https://avatars.githubusercontent.com/in/933528?s=60&v=4){kind=small}
![baz-reviewer-dev[bot]](https://avatars.githubusercontent.com/in/389167?s=60&v=4){kind=small}
yuvalyacoby
approved these changes
Jan 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Generated description
Below is a concise technical summary of the changes proposed in this PR:
graph LR PullRequestCard_("PullRequestCard"):::modified getCIIcon_("getCIIcon"):::modified getReviewStatus_("getReviewStatus"):::modified getReviewStatusDisplay_("getReviewStatusDisplay"):::modified getBazReviewerStatus_("getBazReviewerStatus"):::added getBazBadge_("getBazBadge"):::added isBazReviewer_("isBazReviewer"):::added PullRequestCard_ -- "Unchanged call; CI badge usage remains same." --> getCIIcon_ PullRequestCard_ -- "Now excludes baz reviewers when computing human review status." --> getReviewStatus_ PullRequestCard_ -- "Displays human review status possibly altered by baz exclusion." --> getReviewStatusDisplay_ PullRequestCard_ -- "Newly added call to compute baz-reviewer approval/review state." --> getBazReviewerStatus_ PullRequestCard_ -- "Newly added call to render baz reviewer badge." --> getBazBadge_ getReviewStatus_ -- "Uses isBazReviewer to filter out bot baz-reviewer entries." --> isBazReviewer_ getBazReviewerStatus_ -- "Identifies baz-reviewer reviews to summarize baz approval status." --> isBazReviewer_ classDef added stroke:#15AA7A classDef removed stroke:#CD5270 classDef modified stroke:#EDAC4C linkStyle default stroke:#CBD5E1,font-size:13pxSeparates 'baz-reviewer' from human reviewers by introducing a dedicated status badge and filtering logic in the pull request card. Implements a regex-based identification utility and integrates
vitestfor automated testing of the new reviewer logic.vitestas the testing framework, configure the test environment, and update the CI/CD pipeline to include a test execution step.Modified files (3)
Latest Contributors(2)
isBazReviewerto identify specific bot accounts and updatePullRequestCardto display a separate status badge for these reviews.Modified files (3)
Latest Contributors(1)