Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Alright, sure, but you're picking nits and I'm not sure what your underlying point is.


I do not believe it's nitpicking. I was supporting the post you replied to (https://news.ycombinator.com/item?id=33313036), which claimed that git indeed does not have review tools, and that the review tools/flows are and should be external to git. You claimed otherwise, and I argued that you were incorrect in that claim.

Isn't that what discussion board is for? I agree that this particular topic is not life-and-death important, but still worth discussing, IMHO. If you think otherwise, just walk away, you do not owe anybody anything.


It isn't picking nits, git does patch management, there is nothing and shouldn't be for review.

It matters because where you assign your responsibilities is the difference between spaghetti code/architecture and a well designed one. If you think this is nitpicking, I encourage you to (re)view the general sofware concepts of cohesion and coupling.

(Part of) gits success is it is a tool focused on doing one thing well - I believe Linus is on record as saying the other tooling (commercial/svn) largely failed because they were not really focused on source control i.e. patch management. Its part of the reason he constructed it in the first place, the tool he was using was overcomplex and bad at doing simple things.

Github's success until this point has been largely that it focused on building review infrastructure. It's based on git, giving it flexibility akin to being based on e.g. http, but it's core is review and facilities in support of that e.g. code search. It's a very particular type of review aimed at a specific audience, but it's not git, nor should it be.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: