logoalt Hacker News

jghnyesterday at 6:48 PM2 repliesview on HN

On any project where pre-commit hooks are used, the first thing I do is disable them. What I do when the code is on my side of the line isn't your business.


Replies

benrutteryesterday at 8:01 PM

I agree on the other side of the fence! I quite like precommit when I use it, but I've never imposed it on any of my projects. Some people use commits sporadically then squash down- I really don't feel comfortable breaking someone's personal workflow to that degree.

I almost always have a "this cicd must pass to merge" job, that includes linting etc, and then use squash commits exclusively when merging.

show 1 reply
ncglyesterday at 9:23 PM

Would you add type: ignore to all the files too?

My coworker did that the other day and I'm deciding how to respond.

show 2 replies