The source project of this merge request has been removed.
refactor: cleanup support/ci/shellcheck script
What does this Merge Request do?
Follow up from !1190 (comment 357673279)
Rather than splitting find results via shell, xargs
does it for you and supports (via -print0
/(--null
|-0
) combination) spaces (
), tabs (\t
), newlines (\n
) and other things the shell may break up via IFS
@ashmckenzie - This is the follow up from last night's comment on !1190 (merged)
Merge Request checklist
-
Tests added for new functionality. If not, please raise Issue to follow-up. -
This change is backward compatible. If not, please include steps to communicate to our users. -
Documentation added/updated, if needed. -
gdk doctor
test added, if needed.