Skip to content

Lint updates (batch 2) from reviewdog/action-actionlint#126

Merged
CvH merged 5 commits into
LibreELEC:mainfrom
heitbaum:lint2
Jun 8, 2026
Merged

Lint updates (batch 2) from reviewdog/action-actionlint#126
CvH merged 5 commits into
LibreELEC:mainfrom
heitbaum:lint2

Conversation

@heitbaum

@heitbaum heitbaum commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

- reviewdog/action-actionlint@v1 (SC2001, SC2046, SC2086)
heitbaum added 2 commits June 7, 2026 21:06
- reviewdog/action-actionlint@v1 (SC2086)
SC2046: Quote this to prevent word splitting.
Each line of packages_for_autoupdate.txt is passed as a separate positional
argument to tools/update-scan via intentional word splitting.

- reviewdog/action-actionlint@v1 (SC2046)
… expand

SC2016: Expressions don't expand in single quotes, use double quotes for that.
GraphQL variables ($repo, $headRepo, $owner, $name etc.) use shell-like $ syntax
but are resolved by the GraphQL engine, not the shell. Single quotes are required
to prevent shell expansion.

- reviewdog/action-actionlint@v1 (SC2016)
@heitbaum heitbaum force-pushed the lint2 branch 2 times, most recently from aeeb600 to 37b6538 Compare June 7, 2026 11:14
@heitbaum heitbaum marked this pull request as ready for review June 7, 2026 11:28
@CvH CvH merged commit fb87790 into LibreELEC:main Jun 8, 2026
1 check passed
@heitbaum heitbaum deleted the lint2 branch June 8, 2026 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants