Feedback on pre-push hooks
@sstern and @dennis were exploring the possibility of running eslint, prettier and stylelint concurrently in a pre-push hook in order to:
- Save time by running some frontend static analysis jobs in parallel
- Reduce cost by optimizing pipeline runtime by having these jobs run prior to push and perhaps removed from the pipeline jobs themselves, or relocating the pipeline step to a final job so it's at least not running all the time.
The purpose of this issue is to gather feedback on whether this is a good idea or not. Of course this has impliciations on anyone who is working on GDK, so there are still many details to be worked out, such as only running this pre-push hook when frontend files have been changed, or having some way to bypass them ideally when a WIP branch is being worked on.
Edited by Dennis Tang