-
Notifications
You must be signed in to change notification settings - Fork 45
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
Update weaver dependencies and repository links #457
Conversation
sandeepnRES
commented
May 17, 2024
•
edited
Loading
edited
- chore: update all go module dependencies, add all go tests to workflow
- fix(weaver-go-sdk): use updated weaver protobuf for fabric and corda view
- chore: add script to generate checksums for go modules
- fix: update broken corda repository links
- ci: fix besu asset exchange workflow to use node 16
0af7acc
to
6109b26
Compare
Looks like we are using Node 16 (as per the diff) and not Node 18 (as per the above comment). Right? |
core/network/fabric-interop-cc/libs/testutils/mocks/chaincodestub.go
Outdated
Show resolved
Hide resolved
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.
Just a couple of minor fixes required.
Signed-off-by: Sandeep Nishad <[email protected]>
…view Signed-off-by: Sandeep Nishad <[email protected]>
Signed-off-by: Sandeep Nishad <[email protected]>
Signed-off-by: Sandeep Nishad <[email protected]>
Signed-off-by: Sandeep Nishad <[email protected]>
Yes we are using Node 16, sorry that was old description, just now edited the PR desciption. |