← Back to context

Comment by onion2k

3 hours ago

does that mean that E2E tests that catch bugs are a strong signal that the team isn't doing well?

Yes. E2E tests are there to give you confidence that future changes haven't broken things. They're not there to catch bugs before the feature goes to production. Unit and integration tests should do that though.

You shouldn't just eliminate PR reviews because "we communicate inside the team".

You should eliminate them as soon as they're not giving you any real value, but if you don't eliminate them before that team's will stop trying to get that value in a better way because they believe the PR process catches bugs. It doesn't though, so all it really achieves is stopping the team trying to improve.