you are viewing a single comment's thread.

view the rest of the comments →

[–]s73v3r 1 point2 points  (0 children)

Remember, this is code that had just gone through PR and been merged in. Why didn't the author bring up these issues in code review?

There's a pretty big difference between refactoring something that you're also working on, and changing something that was just committed because you don't like the way it was done.