-
Notifications
You must be signed in to change notification settings - Fork 91
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
Upgrade yq from v3.x to v4.x #721
Upgrade yq from v3.x to v4.x #721
Conversation
Some example scripts still use an older version of yq. To run the example scripts properly newer version of yq is added. Signed-off-by: Deepjyoti-Sarmah <[email protected]>
utils/docker/base-dev/Dockerfile
Outdated
@@ -86,7 +86,7 @@ RUN go install golang.org/x/tools/cmd/goimports@latest \ | |||
&& go install github.com/maxbrunsfeld/counterfeiter/v6@latest \ | |||
&& go install honnef.co/go/tools/cmd/[email protected] \ | |||
&& go install github.com/client9/misspell/cmd/misspell@latest \ | |||
&& go install github.com/mikefarah/yq/v3@latest | |||
&& go get github.com/mikefarah/yq/v4@latest |
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.
I believe this should be go install ...
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.
I thank you pointing out the error and fixing it.
…thub.com/mikefarah/yq/v4@latest" Signed-off-by: Deepjyoti-Sarmah <[email protected]>
Signed-off-by: Deepjyoti-Sarmah <[email protected]>
@chenchanglew may I ask you to look at this PR as well and test fpc samples? |
ya sure let me check it. |
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 @Deepjyoti-Sarmah for this PR! I tested it almost worked. Here some changes.
samples/deployment/fabric-smart-client/the-simple-testing-network/env.sh
Outdated
Show resolved
Hide resolved
Signed-off-by: Deepjyoti-Sarmah <[email protected]>
Signed-off-by: Deepjyoti-Sarmah <[email protected]>
Hey @mbrandenburger thank you for the suggestion. I have made the required changes |
Signed-off-by: Marcus Brandenburger <[email protected]>
Some example scripts still use an older version of yq. To run the example scripts properly newer version of yq is added.
What this PR does / why we need it: Upgrades the yq V3.x to yq v4.x
Which issue(s) this PR fixes:
./samples/deployment/test-network/setup.sh
./samples/deployment/fabric-smart-client/the-simple-testing-network/env.sh
./samples/deployment/test-network/update-connection.sh
./samples/deployment/test-network/fabric-samples/test-network-nano-bash/README.md
./README.md
./utils/docker/dev_peer_cc-builder/Dockerfile
./utils/docker/base-dev/Dockerfile
Fixes #719