Re:ChatGPT powered GitHub code review app
by eikenberry on Tuesday March 07, 2023 @04:05PM
(#35061195)
It has to be an online service until they get the models more refined and hardware advanced enough that you can run these locally. If you can't run the model locally then it might as well be a service you tie into at the service (eg. github) level. That is, IMO, services are most useful with other services.
[ Parent
| on HN ]
Re:ChatGPT powered GitHub code review app
by andiareso on Tuesday March 07, 2023 @04:06PM
(#35061214)
I think they are saying the app could be a pre-commit hook where their API is hit before code goes up to remote
Re:ChatGPT powered GitHub code review app
by eikenberry on Tuesday March 07, 2023 @04:27PM
(#35061505)
Yes. I'm saying that due to this relying on an external service it would be inappropriate for a pre-commit hook as you could only commit while online.
pre-push hook?
Re:ChatGPT powered GitHub code review app
by verdverm on Tuesday March 07, 2023 @04:46PM
(#35061706)
git commit --no-verify ?
Re:ChatGPT powered GitHub code review app
by bottled_poe on Tuesday March 07, 2023 @05:16PM
(#35062031)
If the developer wants to slow their own development workflow by delaying the code review, that’s.. concerning.
Re:ChatGPT powered GitHub code review app
by verdverm on Tuesday March 07, 2023 @07:58PM
(#35063760)
in what way is your comment pertaining to this thread?
If a user is offline, they are not going to be able to push code for review anyway. Either way, it seems prudent to shift validation and checks closer to the developer, before pushing code. Otherwise you end up with a series of commit & push, wait for check, commit & push fixes, wait for check, commit & push more fixes...
> [...] it would be inappropriate for a pre-commit hook as you could only commit while online.
Why does that matter?
Re:ChatGPT powered GitHub code review app
by eurasiantiger on Tuesday March 07, 2023 @05:37PM
(#35062291)
Hey, some of us live in caves.
Add the —no-verify flag when working out of your cave.