ci(lint): add shell linter - Differential ShellCheck#51
ci(lint): add shell linter - Differential ShellCheck#51
Conversation
It performs differential ShellCheck scans and reports results directly on GitHub. documentation: https://github.com/redhat-plumbers-in-action/differential-shellcheck Signed-off-by: Jan Macku <jamacku@redhat.com>
This is false positive, but since it is only ShellCheck defect in this script, let's clean it. Signed-off-by: Jan Macku <jamacku@redhat.com>
|
@jamacku : thank you for offering an idea to improve our workflow CI process! ❤️ In the past, I have used
Putting aside whether all errors should be identified and fixed, we don't have a linting process for the repository, which is largely powered by a shell script, I think this is something that would benefit the workflows here given several key features below:
@williammartin : Any concerns or thoughts about how we might adopt this? |
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
Differential ShellCheck is a GitHub action that performs differential ShellCheck scans on shell scripts changed via PR and reports results directly in PR.
I see that your script is in great shape, but I think that you might find the
differential-shellcheckaction useful. It is able to produce reports in SARIF format. GitHub understands this format and is able to display it nicely as a PR comment, and on theFiles Changedtab, please see below.Documentation is available at @redhat-plumbers-in-action/differential-shellcheck. Let me know If you are missing some feature or option. I'm always happy to extend functionality.