run-local-checks should ignore vendor and node_modules

Ready for feedback. Two things to note

1. I could not find a verbose option for shellcheck
2. The eslint --ignore-pattern argument is supposed to take multiple values, but I could not get that to work, I tried a string comma-separated, space separate, square brackets. But I found that you can added two --ignore_pattern arguments, and the debug shows that this does work and both folders are ignored

Sorry, I meant 'needs review'

Not sure about the --ignore-pattern issue but the alternative syntax seems ok, so there is no need to fight it. And no worries about the verbose mode for shellcheck, I quickly checked the documentation and couldn't find anything either.

I think the MR looks good.

Thank you.

Automatically closed - issue fixed for 2 weeks with no activity.