logoalt Hacker News

WhyNotHugotoday at 4:22 PM1 replyview on HN

Doesn’t this mean that a first review might request that a specific change be reverted, and then a later reviewer reviews that reversion? That’s essentially reviewing a noop, but understanding the it’s a noop requires carefully checking all previous now-invalidated changes.


Replies

pierrekintoday at 4:51 PM

No, each PR is based on the previous one, so the reviewer only needs to consider the ideas that are new in each PR one at a time.