Skip to content
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

ci: Update shellcheck.sh to pass all args to shellcheck #2657

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

marun
Copy link
Contributor

@marun marun commented Jan 24, 2024

This is useful to vary the output format (e.g. -f gcc) for local execution (the default is more detailed but not as machine-consumable e.g. for editor consumption).

@marun marun self-assigned this Jan 24, 2024
@marun marun force-pushed the ci-shellcheck-args branch from 68de7a1 to 6ede30a Compare January 24, 2024 07:53
@marun marun added the ci This focuses on changes to the CI process label Jan 24, 2024
# Try to install a local version
SHELLCHECK="${REPO_ROOT}/bin/shellcheck"
Copy link
Contributor Author

@marun marun Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably preferable to ./build to differentiate artifacts we create vs 3rd party. ./bin is also ignored.

@StephenButtolph StephenButtolph added this to the v1.10.19 milestone Jan 24, 2024
@StephenButtolph StephenButtolph added this pull request to the merge queue Jan 24, 2024
@StephenButtolph StephenButtolph removed this pull request from the merge queue due to a manual request Jan 24, 2024
@StephenButtolph StephenButtolph added this pull request to the merge queue Jan 24, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 24, 2024
@StephenButtolph StephenButtolph added this pull request to the merge queue Jan 25, 2024
Merged via the queue into master with commit 1302ed8 Jan 25, 2024
17 checks passed
@StephenButtolph StephenButtolph deleted the ci-shellcheck-args branch January 25, 2024 18:14
mboben pushed a commit to mboben/avalanchego that referenced this pull request Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This focuses on changes to the CI process
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants