top | item 37720701

(no title)

codeapprove | 2 years ago

I agree 1000%. I’m the creator of what I believe is a better review interface for GitHub (https://codeapprove.com) but there are also many others:

  * CodeApprove (codeapprove.com)
  * Graphite (graphite.dev)
  * Reviewable (reviewable.io)
  * Axolo (axolo.co)
  * Viezly (viezly.com)
  * Mergeboard (mergeboard.com)
  * Codestream (codestream.com)
  * Pullpo (pullpo.io)
  * ReviewPad (reviewpad.com)
  * Planar (useplanar.com)
  * Visibly (visibly.dev)
  * Codelantis (codelantis.com)

I think in the end we should not expect GitHub to provide the best option here. We should expect them to provide a basic option (which they do) and for sophisticated consumers to pay more for a much better option. Everyone should be shopping for code review tools!

discuss

order

noirscape|2 years ago

I disagree on this; GitHub should be building the best option here if they have the resources to do so. The fact that the review interface is as basic as it and so prone to accidentally marking the wrong comments as outdated is a major issue (one that other software like Phorge[0] already shows is possible if we're sticking to the realm of "non-mailing client reliant git servers").

Having to bolt extra features on top of GitHub to make it work properly is a shortcoming of GitHub, it shouldn't be an opportunity to build more tooling the customer has to pay for on top of it. Granted, I can see that conversation would get us nowhere given your income relies on selling people features GitHub is languishing on - you have an obvious interest in keeping that feature shitty.

[0]: Phabricator was disabled in 2022, Phorge is the new fork.

rat9988|2 years ago

"I think in the end we should not expect GitHub to provide the best option here. We should expect them to provide a basic option (which they do) and for sophisticated consumers to pay more for a much better option. Everyone should be shopping for code review tools! "

I understand this linke of thinking might suit you but I fear it is not as convincing as it sounds to you. At least it's not to me.

piotrkaminski|2 years ago

Here's how I like to think about it: GitHub is a generalist. They have a big platform with lots of features besides code review, so even though they also have lots of employees they won't be able to focus on code review as much as a dedicated company could. They also have a huge number of users to please so they can't afford to rock the boat too much or make the learning curve too steep.

I think therefore it's pretty much inevitable that if you need a more advanced code review tool you'll end up picking a third party one. Though admittedly, as the founder of Reviewable, that thinking does rather suit me too. ("It is difficult to get a man to understand something when his salary depends on his not understanding it" and all that. :D )

jayroh|2 years ago

Question: does CodeApprove place related files in closer proximity during review? I would _love_ to have a class and its test next to each other instead of sorted alphabetically. I'm tired of jumping around all over the place, trying to traverse through my review thought process.

fahhem|2 years ago

I can't speak for CodeApprove, but Reviewable has file grouping capabilities so users can groups files based on anything they want (using a javascript function to do it)

codeapprove|2 years ago

CodeApprove does not have that feature because we don't currently do any smart code parsing to understand what would make a file "related". I think Viezly (https://viezly.com/) does the best job at that.

hahn-kev|2 years ago

I feel like if you want that then you should colocate the files in the same folder.

mhh__|2 years ago

That's like saying it's OK to expect (say) Ford to make a car with no steering wheel. GitHub basically define the baseline for the entire industry and have millions upon millions to blow on trying to do better.

(OK, Saab did actually do that once but they're weird)

ravenstine|2 years ago

The two aren't even remotely comparable. Whose compny went out of control and crashed because of GitHub missing a feature with their code review?

jacobegold|2 years ago

I'm keeping an eye on Pierre (https://pierre.co/) as well — not being subject to the whims of GitHub makes for some interesting ideas!

dansiemens|2 years ago

> We should expect them to provide a basic option (which they do) and for sophisticated consumers to pay more for a much better option

GitHub is fairly feature rich imo, _especially_ when compared to something like CodeCommit on AWS. I’ve been forced to use CodeCommit on client engagements and it’s absolutely horrid. Honestly if your tool supported CodeCommit I’d say the value proposition would skyrocket.

piotrkaminski|2 years ago

I'm curious, are you allowed to name any folks using CodeCommit? I thought it wasn't particularly popular.

epolanski|2 years ago

Average org already pays so many saas and I have to pay one for code reviews too when I'm already paying GitHub?

Also, there isn't only paying customers having this issue but large OS too.

jacobegold|2 years ago

Large OSS feels like a different use case than the big company one, to be honest -- I think that what maintainers care about is vastly different than what companies do. GitHub feels optimized for the former (OSS) as it is today.