Skip to content

ci: claude pr review with fix suggestions and opt out - #1861

Open
port19x wants to merge 5 commits into
masterfrom
ci-claude-review
Open

ci: claude pr review with fix suggestions and opt out#1861
port19x wants to merge 5 commits into
masterfrom
ci-claude-review

Conversation

@port19x

@port19x port19x commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Pull Request Template

Type of change

  • Bug fix
  • Feature
  • Documentation update

Description

Apparently this is possible.
I made sure to have it include instructions for committable suggestions in review comments, as that's the highest value automation we can get on top of our shellcheck and shfmt runs.
Contributors who aren't fond of AI are able to opt out of the review by ticking the box at the top of the PR template.

port19x and others added 2 commits August 9, 2026 17:03
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@port19x
port19x marked this pull request as draft August 9, 2026 15:50
port19x and others added 2 commits August 9, 2026 18:05
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@port19x

port19x commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator Author

Demo: port19x/ani-cli#4

@port19x

port19x commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

Two scenarios:
Bad PR, linter breakage and removes stuff: port19x/ani-cli#9
Good PR, literally #1859 that me and @zen.mori are working on: port19x/ani-cli#10

@port19x
port19x marked this pull request as ready for review August 12, 2026 06:32
@port19x

port19x commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

We can also bump it up to Opus 5 Extra if you prefer its output:
Good PR: port19x/ani-cli#17
Bad PR: port19x/ani-cli#18

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