2 ms·
Our code quality checks run as a CI make target instead of a hook. We then have our repo set up to disallow direct push- everything has to go through a PR and t
by dharmab 7y ago
Our code quality checks run as a CI make target instead of a hook. We then have our repo set up to disallow direct push- everything has to go through a PR and therefore CI.
- daveFNbuck 7y agoEither way, it's important to validate the checks on existing code before making them a blocker.
- dharmab 7y agoYes, which is why I like the make target method. You can make the target apply to new files exclusively at first, then incrementally add old files as they are touched in the course of normal work.