Skip to content

New blog post: ai-based code review#1107

Merged
TomasTomecek merged 3 commits intopackit:mainfrom
TomasTomecek:code-review
Feb 24, 2026
Merged

New blog post: ai-based code review#1107
TomasTomecek merged 3 commits intopackit:mainfrom
TomasTomecek:code-review

Conversation

@TomasTomecek
Copy link
Member

You are very welcome to share your perspective with AI-powered code review because I tried to write it as it's a team effort.

@centosinfra-prod-github-app
Copy link
Contributor

Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

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

nice work!


## [Gemini-code-assist](https://github.com/marketplace/gemini-code-assist)

Our AI tool of choice for our upstream reviews is Gemini-code-assist. It's free, easy to set up, and you get to see the comments inside the GitHub interface. Since it's powered by the Gemini models, it can provide solid feedback. If your PR contains clear mistakes, this tool can easily find them.
Copy link
Member

Choose a reason for hiding this comment

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

If your PR contains clear mistakes, this tool can easily find them.

My feeling is that we revealed with it not only clear mistakes, but also more subtle ones, and it definitely prevented lot of bugs from getting into production. But curious what do others think.

Copy link
Member

Choose a reason for hiding this comment

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

Agreed, sometimes it can find very non-obvious issues.

Copy link
Member Author

Choose a reason for hiding this comment

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

amazing feedback, I'll incorporate it


![Gemini referencing the style guide](img/gemini_ref.png)

I'd say the biggest downside of this tool is that you need to open a PR to see the feedback, you cannot run it locally.
Copy link
Member

Choose a reason for hiding this comment

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

I would maybe also mention the related advantage, that it can encourage the team to align on technical debt or complex logic that the tool brings up as a review comment.

@centosinfra-prod-github-app
Copy link
Contributor


This tool can be infinitely helpful, especially if your codebase is easy to navigate, is not colossal, and has solid documentation which Claude can follow.

Several people on the team use Claude Code daily and it is part of our daily job. We've gotten used to it so much that we couldn't live without it, honestly. It's so convenient and helpful.
Copy link
Member

Choose a reason for hiding this comment

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

I'm yet to give it a try 🙈

Copy link
Member Author

Choose a reason for hiding this comment

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

oh true, let me add that info there

@centosinfra-prod-github-app
Copy link
Contributor

@centosinfra-prod-github-app
Copy link
Contributor

For sake of running script `files/generate_usage.py`

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
Assisted-by: Claude
@centosinfra-prod-github-app
Copy link
Contributor

Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

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

🚀

@centosinfra-prod-github-app
Copy link
Contributor


# Our experience with AI-based code review

One amazing benefit that modern LLMs come with is using them as a linter, or a pair programmer. You can easily get feedback on your code with such an approach. If the feedback is solid, your code is improved. If the review is poor, you can just disregard it. But overall with very little effort you can gain a lot.
Copy link
Member

@mfocko mfocko Feb 24, 2026

Choose a reason for hiding this comment

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

You can easily get feedback on your code with such an approach.

Is this supposed to be interpreted as using the LLM?

you can just disregard it

or prompt again…

Copy link
Member Author

Choose a reason for hiding this comment

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

improved the first paragraph to address the first comment, but second is silently implied, we don't need to be pixel-perfect here :)

@TomasTomecek TomasTomecek removed the mergeit Merge via Zuul label Feb 24, 2026
@centosinfra-prod-github-app
Copy link
Contributor

Co-authored-by: Laura Barcziová <49026743+lbarcziova@users.noreply.github.com>
Co-authored-by: Nikola Forró <nforro@redhat.com>
Co-authored-by: Maja Massarini <mmassari@redhat.com>
Co-authored-by: Matej Focko <mfocko@users.noreply.github.com>
Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
Assisted-by: Claude
@centosinfra-prod-github-app
Copy link
Contributor

@TomasTomecek TomasTomecek merged commit 37961e7 into packit:main Feb 24, 2026
4 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Packit pull requests Feb 24, 2026
@TomasTomecek TomasTomecek deleted the code-review branch February 24, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants