Skip to content

refactor(lint): fix shellcheck warnings - #173

Merged
ekalinin merged 1 commit into
masterfrom
refactor/shellcheck-warnings
Sep 28, 2026
Merged

ekalinin merged 1 commit into
masterfrom
refactor/shellcheck-warnings

Conversation

@ekalinin

Copy link
Copy Markdown
Owner

Summary

Fixes #167 together with #172: make lint passes once both are merged.

  • SC2006 (9 places): legacy backticks replaced with $(...). None of them contain
    backslashes, so the result is the same.
  • SC2016 (2 places): # shellcheck disable=SC2016 above each awkscript='{...}'
    assignment. The awk scripts are single-quoted on purpose so that $0 reaches awk
    unexpanded. The directive is set per assignment, not on the whole if block.

SC2181 is a real bug rather than a style issue and is fixed separately in #172.

No behavior changes.

Testing

@ekalinin
ekalinin merged commit 1c9e38a into master Sep 28, 2026
1 check passed
@ekalinin
ekalinin deleted the refactor/shellcheck-warnings branch September 28, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

make lint fails on shellcheck style and info warnings

1 participant