fix shell errors from shellcheck #4032

Merged
merged 1 commit into from Oct 12, 2017

Conversation

Projects
None yet
5 participants
Contributor

albertodonato commented Oct 12, 2017

No description provided.

fix shell errors from shellcheck
Signed-off-by: Alberto Donato <alberto.donato@gmail.com>

zyga approved these changes Oct 12, 2017

Looks good, thank you for the PR

Lgtm, thanks for the improvements..

codecov-io commented Oct 12, 2017

Codecov Report

Merging #4032 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4032      +/-   ##
==========================================
- Coverage   75.88%   75.88%   -0.01%     
==========================================
  Files         431      431              
  Lines       36915    36915              
==========================================
- Hits        28014    28012       -2     
- Misses       6948     6950       +2     
  Partials     1953     1953
Impacted Files Coverage Δ
overlord/ifacestate/helpers.go 62.33% <0%> (-0.67%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ea3421...63b693b. Read the comment docs.

@stolowski stolowski merged commit d5f9c75 into snapcore:master Oct 12, 2017

7 checks passed

artful-amd64 autopkgtest finished (success)
Details
artful-i386 autopkgtest finished (success)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
xenial-amd64 autopkgtest finished (success)
Details
xenial-i386 autopkgtest finished (success)
Details
xenial-ppc64el autopkgtest finished (success)
Details
zesty-amd64 autopkgtest finished (success)
Details

@albertodonato albertodonato deleted the albertodonato:shellcheck-lints branch Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment