Skip to content

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Sep 16, 2025

The "liskin/gh-problem-matcher-wrap" action is used in the "Check Shell" workflow to enhance the workflow run results summary by applying a "problem matcher" to the output of the shellcheck tool.

This action is a wrapper around shell commands, so we would expect its run input to work the same as the jobs.<job_id>.steps[*].run key of the GitHub Actions workflow. However, this is not the case. The commands specified via the jobs.<job_id>.steps[*].run key can be split into multiple lines using the standard shell line continuation operator. That is useful in the case of long/complex shell commands such as the one used to execute the shell:check task in this workflow step. This is not possible when using "liskin/gh-problem-matcher-wrap".

A comment is added to explain this behavior.

…-matcher-wrap" action

The "liskin/gh-problem-matcher-wrap" action is used in the "Check Shell" workflow to enhance the workflow run results
summary by applying a "problem matcher" to the output of the shellcheck tool.

This action is a wrapper around shell commands, so we would expect its `run` input to work the same as the
`jobs.<job_id>.steps[*].run` key of the GitHub Actions workflow. However, this is not the case. The commands specified
via the `jobs.<job_id>.steps[*].run` key can be split into multiple lines using the standard shell line continuation
operator. That is useful in the case of long/complex shell commands such as the one used to execute the `shell:check`
task in this workflow step. This is not possible when using "liskin/gh-problem-matcher-wrap".

A comment is added to explain this behavior.
@per1234 per1234 self-assigned this Sep 16, 2025
@per1234 per1234 added topic: documentation Related to documentation for the project type: enhancement Proposed improvement labels Sep 16, 2025
@per1234 per1234 merged commit 93724de into arduino:main Sep 16, 2025
21 checks passed
@per1234 per1234 deleted the comment branch September 16, 2025 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: documentation Related to documentation for the project type: enhancement Proposed improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant