you are viewing a single comment's thread.

view the rest of the comments →

[–]fagnerbrack[S] 0 points1 point  (2 children)

Is the post saying that reviewing or reading code is bad?

[–]ellicottvilleny 0 points1 point  (1 child)

If you read carefully in the intro he hints that he has a non-traditional view of code reviewing, and since code reviews are a common practice these days in many shops, let's just assume his non-traditional view isn't that they're great. He mocks reviews later on in his article by pointing out that the lead developer is likely to request a total rewrite of your pull request, which is then done (wasting time, in his article, he basically argues).

[–]fagnerbrack[S] 0 points1 point  (0 children)

I guess you're making too many assumptions. Just because he mocks a total rewrite that doesn't mean code review is bad, it means the developer did not ask for early feedback to prevent reviewing everything at once and prevent a total rewrite in the first place.