mrtazz / checkmake

experimental linter/analyzer for Makefiles
MIT License
1.03k stars 45 forks source link

Adds a system-provided checkmake hook for pre-commit #93

Closed colindean closed 1 year ago

colindean commented 1 year ago

Problems with pre-commit's invocation of go install had me wanting more options for how to run hooks in case the automated installation fails. Since checkmake is available in at least Homebrew, it'd be nice to have a hook that can use a checkmake executable already installed in the system PATH.

Checklist

Not all of these might apply to your change but the more you are able to check the easier it will be to get your contribution merged.

mrtazz commented 1 year ago

thanks for taking the time to contribute!