Kissaki@programming.dev to Programming@programming.devEnglish · 5 days agoThe Pain That is GitHub Actions - Feldera Blogwww.feldera.comexternal-linkmessage-square25fedilinkarrow-up139arrow-down19
arrow-up130arrow-down1external-linkThe Pain That is GitHub Actions - Feldera Blogwww.feldera.comKissaki@programming.dev to Programming@programming.devEnglish · 5 days agomessage-square25fedilink
minus-squareFizzyOrange@programming.devlinkfedilinkarrow-up5arrow-down1·4 days agoTo check that people ran the pre-commit linters. Committing itself won’t be possible That’s not how pre-commit hooks work. They’re entirely optional and opt-in. You need CI to also run them to ensure people don’t forget.
minus-squarelemmeBe@sh.itjust.workslinkfedilinkarrow-up1arrow-down3·4 days agoThey’re optional if you make them optional. I didn’t. You do as you please. 😄
minus-squareFizzyOrange@programming.devlinkfedilinkarrow-up5·4 days agoNo, they’re inherently optional in Git. There’s no way to “check in” a git hook. You have to put in your README Clone the repo and then please run pre-commit install! Oh and whatever you do don’t git commit --no-verify! You definitely need to actually check the lints in CI. It’s very easy though, just add pre-commit run -a to your CI script.
minus-squarebamboo@lemm.eelinkfedilinkarrow-up2·4 days agopre-commit also has a free service for open source GitHub repos too. They’ll even push an autofix commit for you if your tools are configured for it
To check that people ran the pre-commit linters.
That’s not how pre-commit hooks work. They’re entirely optional and opt-in. You need CI to also run them to ensure people don’t forget.
They’re optional if you make them optional. I didn’t. You do as you please. 😄
No, they’re inherently optional in Git. There’s no way to “check in” a git hook. You have to put in your
READMEYou definitely need to actually check the lints in CI. It’s very easy though, just add
pre-commit run -ato your CI script.pre-commit also has a free service for open source GitHub repos too. They’ll even push an autofix commit for you if your tools are configured for it