Comment

Comment by williamdclt

As a reviewer, —amend means I need to reread the whole commit to see what changed. A tiny commit means it takes me seconds.

My opinion is that a PR should be small enough that it’s desirable to have it squashed into a single commit at the end anyway.

parent williamdclt 2 years ago

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton