Skip to content

Conversation

@michielbdejong
Copy link
Member

No description provided.

@michielbdejong michielbdejong merged commit c68e7b1 into main Sep 18, 2023
@Potherca
Copy link
Member

This violates ShellChecks SC2006 imparative.

If $(...) command substitution does not work in an environment, we might be better served by resolving the environment instead of degrading the quality of the shell script.

For instance, MacOS has the tendency to ship older (outdated) versions of BASH (v3 or even v2).

This issue might be resolved by updating the used bash version.

function teardown {
docker stop "$(docker ps --filter network=testnet -q)"
docker rm "$(docker ps --filter network=testnet -qa)"
docker stop `docker ps --filter network=testnet -q`
Copy link
Member

Choose a reason for hiding this comment

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

👎

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants