Skip to content

Shell scripts: remove bashisms and fix bare variables#38527

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:bash_the_bashisms
Jan 11, 2019
Merged

Shell scripts: remove bashisms and fix bare variables#38527
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:bash_the_bashisms

Conversation

@thaJeztah
Copy link
Member

Reverts #38424, and re-applies the non-double-bracket-changes-etcetera

…otes"

This reverts commit 297b30d.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
This makes my IDE a bit more silent :-)

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

ping @tianon 👍 ❤️

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍 thank you!

@olljanat
Copy link
Contributor

olljanat commented Jan 10, 2019

(reserved for my derek commands)

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💃
@tianon 🤗

@codecov
Copy link

codecov bot commented Jan 11, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@b4842cf). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #38527   +/-   ##
=========================================
  Coverage          ?   36.66%           
=========================================
  Files             ?      608           
  Lines             ?    45174           
  Branches          ?        0           
=========================================
  Hits              ?    16562           
  Misses            ?    26329           
  Partials          ?     2283

@thaJeztah
Copy link
Member Author

💚

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.

6 participants