wide-midnight-78598
04/13/2024, 3:12 AMpants fix fmt lint check test --all-changed
Or maybe two:
foo = "fix fmt lint check --all-changed"
bar = "foo test --all-changed"
I was just updating some documentation and noticed that we call out to the pre-commit hook, instead of just, like, an alias that would not only check, but fix toohappy-kitchen-89482
04/14/2024, 2:38 AMgreen as an alias for thishappy-kitchen-89482
04/14/2024, 2:38 AMwide-midnight-78598
04/14/2024, 4:43 AMpants tidywide-midnight-78598
04/14/2024, 4:44 AMmend maybe, if we're excluding testing 🤷wide-midnight-78598
04/14/2024, 4:46 AMbroad-processor-92400
04/15/2024, 1:19 AMlocal as an alias for this in my work repos:
[cli.alias]
local = "tailor update-build-files fmt lint check ::"
tidy is nice, but could also be pre-push-checks or something descriptive.
I agree with not including test with the default one because that'll often be pretty slow with the --changed-dependents=transitive, I imagine.wide-midnight-78598
04/15/2024, 1:35 AMcurved-manchester-66006
04/15/2024, 1:52 PM[cli.alias]
green = "fmt fix lint check"
--since-main = "--changed-since=origin/main --changed-dependents=transitive"
We don't includ test because we rely on a tag scheme for those and I don't think there is a way to have one command with multiple goals where the tags vary per goal.wide-midnight-78598
04/15/2024, 2:33 PMgreen doesn’t resonate with me - it doesn’t feel like it’ll perform an action which might require manual intervention (e.g. fixing lints that can’t be auto-fixed)