Skip to content

refactor: cleanup support/ci/shellcheck script

Rob Paisley requested to merge (removed):follow-up-from-MR1190 into master

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.

Merge request reports

Loading