← Back to context Comment by bawolff 2 months ago Well we are in a discussion about pre-commit hooks. Pre-commit hooks run on local wip commits. 7 comments bawolff Reply thomashabets2 2 months ago Well, unless you inhibit them with `-n`. Which I would for WIP commits. badgersnake 2 months ago Then what’s the point? Just leave them off and run the tests when you want to run them. thomashabets2 2 months ago Because 99% of my commits are not WIP commits. So I almost always want to run them.Hell, even most WIP commits will pass the tests (e.g. tests are not yet added for the new code), so I'd run them then too. 4 replies →
thomashabets2 2 months ago Well, unless you inhibit them with `-n`. Which I would for WIP commits. badgersnake 2 months ago Then what’s the point? Just leave them off and run the tests when you want to run them. thomashabets2 2 months ago Because 99% of my commits are not WIP commits. So I almost always want to run them.Hell, even most WIP commits will pass the tests (e.g. tests are not yet added for the new code), so I'd run them then too. 4 replies →
badgersnake 2 months ago Then what’s the point? Just leave them off and run the tests when you want to run them. thomashabets2 2 months ago Because 99% of my commits are not WIP commits. So I almost always want to run them.Hell, even most WIP commits will pass the tests (e.g. tests are not yet added for the new code), so I'd run them then too. 4 replies →
thomashabets2 2 months ago Because 99% of my commits are not WIP commits. So I almost always want to run them.Hell, even most WIP commits will pass the tests (e.g. tests are not yet added for the new code), so I'd run them then too. 4 replies →
Well, unless you inhibit them with `-n`. Which I would for WIP commits.
Then what’s the point? Just leave them off and run the tests when you want to run them.
Because 99% of my commits are not WIP commits. So I almost always want to run them.
Hell, even most WIP commits will pass the tests (e.g. tests are not yet added for the new code), so I'd run them then too.
4 replies →