-
Notifications
You must be signed in to change notification settings - Fork 85
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.
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
Fix shell scripts to pass linter #563
Conversation
I haven't had a chance to investigate why, but this PR currently seems to break the docker_e2e.sh integration test (verified on local machine as well) |
fca4d24
to
5234d76
Compare
The PR was targeting the I updated the PR to target I had to add a filter to the linter to skip checks of the shell script in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this! A few comments below.
* [cleanup] Fix shell scripts to pass linter (#563) * Fix shell scripts to pass linter * Skip shell linter on submodule astm-utm * Fix #563 comments - unused variables * Fix #563 comments - boostrap script * [ci] azure-pipelines: Add linter to build step (#576) * azure-pipelines: Add linter to build step * azure-pipelines: Revert to preserve job labels and pipeline name * [deployment] Remove outdated files (#581) * test cases updated with delete in the end. * Script to run prober tests on local DSS instance * Prober test run on local dss instances * comments incorporated * removed outdated files * Update test_constraints_with_subscriptions to support 0.3.17 Co-authored-by: Michael Barroco <michael@orbitalize.com> Co-authored-by: Pratibha <40968555+pratibhagupta2109@users.noreply.github.com>
This pull request fixes shell scripts in order to pass shellcheck linter test.
Test command:
Please note that there are few TODO comments to be discussed with reviewers.