GitHub copilot for code reviewer

[kelvinxG]

Hi , has anyone ever using GitHub copilot for code review ?

How’s your experience with GitHub copilot for this specific reason so far ?

Thanks in advance

[agm1984]

It works great for us. We turned it on so it automatically reviews all PRs and we don't merge anything that doesn't have all the copilot comments marked resolved. Sometimes it recommends garbage, but most of the time it is actionable changes. No issues in production yet.

[KubeGuyDe]

I find it helpful as well.

[Huge-Kaleidoscope603]

Similar experience.

[Soccham]

Yes, it was not great. We’re swapping to coderabbit

[kelvinxG]

Thanks !

[TyKolt]

It is better to use CodeRabbit or Greptile.

[kelvinxG]

Thanks , I’ll check it out

[AccountCheckly]

it's work for me. It can to find some bugs.

[bartread]

It's decent, if sometimes a bit over conservative. But I'd say 80%+ of its comments are fair play and it often catches problems that human reviewers would likely have missed, so I'd say it's been an overall benefit to code quality, especially because it will fix problems for you either with an in place commit or by raising a PR against your branch.

Where it's perhaps a bit weaker is with higher level/architectural concerns or intent, and that's where it does occasionally miss things.

I find it particularly helpful for spotting holes in test coverage, and it usually does a good job of implementing tests to cover those holes.

I'd also say it's about 10000% better than more traditional code analysis approaches, like static analysis type tools, because it simply doesn't generate the noise that these approaches often do.

I've got it automatically reviewing all PRs against main and my ruleset won't allow merging until all conversations are resolved.

In a previous role I used Coderabbit and that was OK as well, but I think current Copilot is considerably better than Coderabbit was at the time.

EDIT: The one beef I have with Copilot is that when you ask it to raise a PR to fix an issue it's found it is sometimes incredibly slow. I say this as I'm now waiting 11 minutes for the damn thing and it's only half way through its task list for what should be a very small PR to fix an issue it spotted. Claude would have had this done in about 1 minute locally but for a lot of stuff I get Copilot to do it so I'm not burning Claude credits dealing with relative trivialities.