Live data from Hacker News

Code Review from the Command Line (2018)

blog.jez.io

1–10 of 11 posts

Re: Code Review from the Command Line (2018)

#2
I’ve gotten used to Vimium [0] (a chrome extension that emulates Vim commands in the browser) and I rarely use the mouse or arrow keys now. For me, that has been the biggest leap in productivity for my CRs.

I agree that it’s nice to review within the context of the terminal, but I still think the UI (at least for Github) is easygoing and productive (specially when you ditch the mouse).

[0] - https://chrome.google.com/webstore/detail/vimium/dbepggeogba...

Re: Code Review from the Command Line (2018)

#4
post #2

I’ve gotten used to Vimium [0] (a chrome extension that emulates Vim commands in the browser) and I rarely use the mouse or arrow keys now. For me, that has been the biggest leap in productivity for my CRs. I agree that it’s nice to review within the context of the terminal, but I still think the UI (at least for Github) is easygoing and productive (specially when you ditch the mouse). [0] - https://chrome.google.com…

We implemented Vim like shortcuts for github enterprise code review screen and it works pretty nice, no mouse required. However I like the dependency graph, maybe we implement it too...

Re: Code Review from the Command Line (2018)

#6
I've been using GitHub's code review features a bit recently. Something I really like is the ability for a reviewer to quickly make suggested code changes that the reviewee can can approve with a button push, instantly committing them. This saves a lot of time, especially for small, relatively insignificant changes.

Overall though I find GitHub's code review a bit fiddly and awkward, and it's features aren't "easily discoverable". Using the feature I mentioned above as an example, it's rare that reviewees actually know about it, or see and use the "Approve" button.

Re: Code Review from the Command Line (2018)

#7
post #6

I've been using GitHub's code review features a bit recently. Something I really like is the ability for a reviewer to quickly make suggested code changes that the reviewee can can approve with a button push, instantly committing them. This saves a lot of time, especially for small, relatively insignificant changes. Overall though I find GitHub's code review a bit fiddly and awkward, and it's features aren't "easily…

To your first point - how? I didn't know you could do this.

Re: Code Review from the Command Line (2018)

#8
post #6

I've been using GitHub's code review features a bit recently. Something I really like is the ability for a reviewer to quickly make suggested code changes that the reviewee can can approve with a button push, instantly committing them. This saves a lot of time, especially for small, relatively insignificant changes. Overall though I find GitHub's code review a bit fiddly and awkward, and it's features aren't "easily…

To your first point - how? I didn't know you could do this.

Comment on a line using this block:

```suggestion # your changes ```

Re: Code Review from the Command Line (2018)

#10
This is a nice workflow - thanks for sharing!

We're using Bitbucket, not GitHub, but this would mostly work there too.

The one thing I was hoping to see and didn't was adding review comments from the command line. It's appealing to review changes in the terminal, but if I have to open another tool to comment the utility drops.

Post reply on HN