Remix.run Logo
mjd 12 hours ago

I agree. I decided years ago that that was a lot of work for little or no benefit.

It's enough for the tests to pass at each merge point.

baq 8 hours ago | parent [-]

…and that’s why squash merge should be the default setting in PRs.

eru 7 hours ago | parent [-]

Yes, it should be the default, but ideally you have the option of preserving history (for PRs where that makes sense) and then your CI/CD should also check that the individual commits build and pass tests.

In general, your CI/CD should make sure that each commit that appears in the 'public' history of main builds and passes tests.