prr icon indicating copy to clipboard operation
prr copied to clipboard

Review individual commits

Open krobelus opened this issue 2 years ago • 6 comments

Instead of reviewing the sum of all changes, I'd like to review commits one-by-one.

Unfortunately, the github API doesn't make this easy. I have a hacky script that tries to compute the line numbers that github expects but it fails when there are merge commits

krobelus avatar Jan 11 '23 18:01 krobelus

Yeah, per commit reviewing would be nice. Although I believe even GH itself has restrictions on what kind of comments you can leave on per-commit reviews. IIRC you can't do spans (someone should double check me)

danobi avatar Jan 13 '23 06:01 danobi

I am also thinking about this in the context of https://github.com/danobi/prr/pull/47#issuecomment-1878953383 . I believe we can build on top of PR metadata for this

danobi avatar Jan 06 '24 17:01 danobi

the hardest part is reverse engineering the github API. An obvious workaround is to post the literal (per commit) prr-files as comments on github, I wonder how well that is received

krobelus avatar Jan 24 '24 17:01 krobelus

the hardest part is reverse engineering the github API

It might be easier to look at prior art. As a reference, the octo.nvim plugin gained support for individual commit reviewing in this PR: https://github.com/pwntester/octo.nvim/pull/294 after seeing it had been solved by https://github.com/ldelossa/gh.nvim

rmacklin avatar Aug 08 '24 16:08 rmacklin

Thanks for the pointers!

danobi avatar Aug 09 '24 16:08 danobi

You're welcome! Thank you for working on this project :) - I'm not yet using prr myself (mainly because I kinda rely on the ability to review individual commits) but it looks quite interesting/promising!

rmacklin avatar Aug 09 '24 22:08 rmacklin