-
Notifications
You must be signed in to change notification settings - Fork 285
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
test(geth-test-ledger): add basic tests #2906
test(geth-test-ledger): add basic tests #2906
Conversation
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 had a couple of quick questions but LGTM overall!
This PR/issue depends on:
|
ccca00f
to
ab080ef
Compare
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.
LGTM (already approved by Peter from our organisation)
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.
LGTM
d04fb6b
to
5742f67
Compare
- Add test suite for geth-test-ledger package. - Add new test suit to CI - It was initially proposed in hyperledger-cacti#2588, I've added some cleanups and improvements. - Run codegen, update missing deps, sort package.json, fix type in socketio-test-setup-helpers, to fix some CI. Closes: hyperledger-cacti#2579 Co-authored-by: Tomasz Awramski <[email protected]> Signed-off-by: Michal Bajer <[email protected]>
5742f67
to
3827001
Compare
Closes: #2579
Depends on #2879
Pull Request Requirements
upstream/main
branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.-s
flag when usinggit commit
command. You may refer to this link for more information.Character Limit
A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.