Skip to content

Conversation

@sophuric
Copy link
Contributor

@sophuric sophuric commented Sep 20, 2022

  • Instead of cat foo | bar, use bar < foo
  • Instead of printf %s "${foo}" | bar, use bar <<< "${foo}"

This also fixes the issues in shellcheck, so I've removed the lines that disable shellcheck

@sophuric sophuric changed the title Remove unnecessary use of cat and printf Remove unnecessary use of cat, printf, and base64 Sep 20, 2022
@CallMeEchoCodes CallMeEchoCodes merged commit 0424575 into CallMeEchoCodes:main Sep 20, 2022
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.

2 participants