Skip to content
This repository has been archived by the owner on Jun 25, 2023. It is now read-only.

shellcheck errors in test.sh #94

Closed
jkirk opened this issue Mar 5, 2022 · 2 comments
Closed

shellcheck errors in test.sh #94

jkirk opened this issue Mar 5, 2022 · 2 comments

Comments

@jkirk
Copy link
Contributor

jkirk commented Mar 5, 2022

We use the Github Action Super-Linter to lint the code base and found some errors in test.sh. You might want to consider fixing it, to make shellcheck happy. 馃槈 Thx!

2022-03-04 12:54:22 [INFO]   File:[/github/workspace/roles-generic/jnv.unattended-upgrades/tests/test.sh]
2022-03-04 12:54:22 [ERROR]   Found errors in [shellcheck] linter!
2022-03-04 12:54:22 [ERROR]   Error code: 1. Command output:
------

In /github/workspace/roles-generic/jnv.unattended-upgrades/tests/test.sh line 7:
red=''
^-^ SC2034 (warning): red appears unused. Verify use (or export if used externally).


In /github/workspace/roles-generic/jnv.unattended-upgrades/tests/test.sh line 8:
green=''
^---^ SC2034 (warning): green appears unused. Verify use (or export if used externally).


In /github/workspace/roles-generic/jnv.unattended-upgrades/tests/test.sh line 9:
neutral=''
^-----^ SC2034 (warning): neutral appears unused. Verify use (or export if used externally).

For more information:
  https://www.shellcheck.net/wiki/SC2034 -- green appears unused. Verify use ...
@jnv
Copy link
Owner

jnv commented Mar 7, 2022

Thanks but these are non-issues, those unused variables are there for output coloring purposes. I can make shellcheck to ignore them though.

@jnv
Copy link
Owner

jnv commented May 23, 2022

I am closing this issue since this role has been deprecated. Feel free to suggest alternatives in #98.

@jnv jnv closed this as not planned Won't fix, can't repro, duplicate, stale May 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants