Skip to content

Experimental Gemini Code Assist configuration - #9273

Open
tlively wants to merge 2 commits into
mainfrom
gemini-review
Open

tlively wants to merge 2 commits into
mainfrom
gemini-review

Conversation

@tlively

@tlively tlively commented Oct 16, 2026

Copy link
Copy Markdown
Member

Add configuration for the Gemini Code Assist GitHub app to experiment with automatic code reviews. The hope is that this will especially help with initial feedback for PRs from new contributors. Also add code review instructions in .gemini/styleguide.md. These instructions are derived from our code review history and will help guide the reviews from GCA.

Add configuration for the Gemini Code Assist GitHub app to experiment with automatic code reviews. The hope is that this will especially help with initial feedback for PRs from new contributors. Also add code review instructions in .gemini/styleguide.md. These instructions are derived from our code review history and will help guide the reviews from GCA.
@tlively
tlively requested a review from a team as a code owner October 16, 2026 22:29
@tlively
tlively requested review from kripken and sbc100 October 16, 2026 22:29
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Comment thread .gemini/styleguide.md Outdated
* Reuse existing helpers in `src/ir/` (`Properties`, `Bits`, `ChildLocalizer`,
`ModuleUtils`, `TableUtils`, `MemoryUtils`, `LUBFinder`,
`SubtypingDiscoverer`, `Match` in `src/ir/match.h`) instead of duplicating
logic.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is a lot of detail here. Do we need it? Maybe we can try without it and add as we go? Or did you derive this list that way somehow?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I told Gemini something like "use gh to get our entire review history and come up with a guide based on that." I have no idea how effective any particular part of the guide is, or what the optimal level of detail is. Since this isn't meant for human consumption, I'm not too concerned about having this be a large grab bag of information.

Starting with no guidance here and then trying to add it back in if we think the reviews aren't very useful sounds like a reasonable strategy. Alternatively, we could err on the side of providing more information to try to get the most useful reviews out of the gate; I doubt removing information would make the reviews better.

Happy to go with whatever your preference is.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've removed it for now.

@sbc100

sbc100 commented Oct 17, 2026

Copy link
Copy Markdown
Member

lgtm if it works!

This branch has not been deployed

No deployments
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.

3 participants