-
Notifications
You must be signed in to change notification settings - Fork 47
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(store): Implement store tests (1/2) #2235
Conversation
You can find the image built from this PR at
Built from eaa3c23 |
055b69c
to
ba2f9bb
Compare
feb40f3
to
6bcab97
Compare
6bcab97
to
a172987
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! Thanks for it!
Just added a couple of comments. The most relevant IMHO is to put the rpc.nim
change in a separate PR.
Cheers
@Ivansete-status I see your point, and I don't really know what it should be. But I do see the usefulness in using 0 as meaning "give me everything" as the type is Maybe we should page an RFC guy? |
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.
Overall looks great! Thanks so much!
699ca20
to
7a796c2
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! Thanks for it!
61d9e29
to
4b32cf4
Compare
f56ceb4
to
fdc2486
Compare
1ca30ae
to
01214b1
Compare
fdc2486
to
c4f8834
Compare
c4f8834
to
e20b308
Compare
* Implement remaining store tests.
Description
Implement store tests
Changes