cremalab/app-web

CI tests on prepush feels like an anti-pattern

luiz-chagas opened this issue · 0 comments

Is your feature request related to a problem? Please describe.
Running CI gets expensive (takes minutes) very quickly and enforcing it before pushing code to a branch discourages the practice of "commit early and often". CI runs on GitHub for free and is required before anything can be merged anyway, so I don't see many advantages of enforcing CI before every push.

Describe the solution you'd like
We could be running npx lint-staged at a minimum so we are still pushing lint-safe code while keeping our wait time down to just seconds instead of minutes. Other errors will happen within GitHub CI and that feels totally fine for me.