Add way to customize shellcheck invocation about actionlint

[rhysd]

Yes, there is no way to modify arguments passed to shellcheck. What arguments do you want to pass and can I ask the reason?

[nijel]

I need to disable SC2046 as nested function generates command line parameters, and we need word splitting there.

[nijel]

The best solution would be to disable it per command (something like https://github.com/koalaman/shellcheck/wiki/Ignore#ignoring-one-specific-instance-in-a-file), but having a global override would be good enough.

[rhysd]

I don't think SC2046 should be always suppressed. But as the document says, it has some rare case where it should be ignored.

I'll consider to override shellcheck arguments by options.

[rhysd]

@nijel Before considering the details of the feature, I want to confirm one thing.

Is -ignore option sufficient for your use case?

actionlint -ignore 'SC2046:'

[nijel]

We currently use actionlint via GitHub super-linter and that doesn't seem to support passing command line parameters, it can only use the configuration file.

[rhysd]

that doesn't seem to support passing command line parameters

That sounds a problem of super-linter. Can we improve super-linter side? I'll take a look at the repo later.

[nijel]

All other linters use configuration files, what is really preferred way - the configuration sits in the repository and you can run linter without any parameters.

[rhysd]

OK, I understood the situation. I'll consider how to handle this.

[rhysd]

I have made PR at super-linter, which allows to pass additional command line options through an environment variable.

github/super-linter#1853

Let's see if this is merged.

[rhysd]

This is not an issue of actionlint and will be fixed github/super-linter#1667. Please comment there.