Skip to content

Clean more shellcheck complaints #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
Sep 8, 2017
Merged

Clean more shellcheck complaints #53

merged 7 commits into from
Sep 8, 2017

Conversation

unixorn
Copy link
Owner

@unixorn unixorn commented Sep 7, 2017

Description

More minor shellcheck fixes to chip away at the code climate errors.

Type of changes

  • A helper script
  • A link to an external resource like a blog post

Checklist:

  • All new and existing tests pass.
  • Any scripts added use #!/usr/bin/env interpreter instead of potentially platform-specific direct paths (#!/bin/sh is an ok exception)\
  • Scripts are marked executable
  • I have confirmed that the link(s) in my PR are valid.
  • I have read the CONTRIBUTING document.
@unixorn unixorn merged commit 9dd8471 into master Sep 8, 2017
@unixorn unixorn deleted the clean-more-shellcheck-complaints branch September 8, 2017 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@unixorn