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

Jv rox 21500 long running clusters should be able to run any commit #42

Open

JoukoVirtanen wants to merge 20 commits into main from jv-ROX-21500-long-running-clusters-should-be-able-to-run-any-commit

Open

Jv rox 21500 long running clusters should be able to run any commit #42

Removed extra braces

GitHub Actions / shellcheck failed Dec 28, 2023 in 1s

reviewdog [shellcheck] report

reported by reviewdog 🐶

Findings (2)

release/start-acs/start-acs.sh|35 col 13| Iterating over ls output is fragile. Use globs.
release/start-acs/start-acs.sh|35 col 13| Use $(...) notation instead of legacy backticks ....

Filtered Findings (0)

Annotations

Check failure on line 35 in release/start-acs/start-acs.sh

github-actions / shellcheck

[shellcheck] release/start-acs/start-acs.sh#L35 <ShellCheck.SC2045>

Iterating over ls output is fragile. Use globs.

Raw output


            ./release/start-acs/start-acs.sh:35:13: error: Iterating over ls output is fragile. Use globs. (ShellCheck.SC2045)

Check notice on line 35 in release/start-acs/start-acs.sh

github-actions / shellcheck

[shellcheck] release/start-acs/start-acs.sh#L35 <ShellCheck.SC2006>

Use $(...) notation instead of legacy backticks `...`.

Raw output


            ./release/start-acs/start-acs.sh:35:13: info: Use $(...) notation instead of legacy backticks `...`. (ShellCheck.SC2006)

View more details on GitHub Actions