Skip to content

ci/lint: Fix lint-shell.sh in CI - #4489

Merged
UdjinM6 merged 4 commits into
dashpay:developfrom
UdjinM6:fix_linter_p2
Oct 10, 2021
Merged

ci/lint: Fix lint-shell.sh in CI#4489
UdjinM6 merged 4 commits into
dashpay:developfrom
UdjinM6:fix_linter_p2

Conversation

@UdjinM6

@UdjinM6 UdjinM6 commented Oct 5, 2021

Copy link
Copy Markdown

Second part of #4405 with actual fixes only.

Fixes things like https://gitlab.com/dashpay/dash/-/jobs/1652527761#L57

@UdjinM6 UdjinM6 added this to the 18 milestone Oct 5, 2021
@UdjinM6
UdjinM6 marked this pull request as ready for review October 5, 2021 23:13

@PastaPastaPasta PastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

utACK for squash merge

@UdjinM6
UdjinM6 merged commit 77b2cfc into dashpay:develop Oct 10, 2021
pravblockc pushed a commit to pravblockc/dash that referenced this pull request Nov 18, 2021
* lint: Skip shell linting if gawk is not installed

* lint: Skip Gitian descriptor scripts checking if jq is not installed

* ci: Install gawk and jq

`yq` requires `jq`

* Fix shellcheck warnings
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jun 17, 2023
* lint: Skip shell linting if gawk is not installed

* lint: Skip Gitian descriptor scripts checking if jq is not installed

* ci: Install gawk and jq

`yq` requires `jq`

* Fix shellcheck warnings
gades pushed a commit to piratecash/pirate that referenced this pull request Dec 9, 2023
* lint: Skip shell linting if gawk is not installed

* lint: Skip Gitian descriptor scripts checking if jq is not installed

* ci: Install gawk and jq

`yq` requires `jq`

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants