-
Notifications
You must be signed in to change notification settings - Fork 17
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
extract test utils to go-unixfsnode #231
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #231 +/- ##
==========================================
- Coverage 71.04% 70.22% -0.82%
==========================================
Files 68 66 -2
Lines 6081 5824 -257
==========================================
- Hits 4320 4090 -230
+ Misses 1520 1501 -19
+ Partials 241 233 -8
|
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 looking at the last commit here, which I believe is the actual diff / difference you want here. It looks fine.
15fd9d3
to
c89b7e3
Compare
merged #232 into this, renamed it to reflect what we're doing now, recreated this branch on top of the new |
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 but @rvagg can you tag unixfsnode 1.6.1 and update before you merge?
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 but @rvagg can you tag unixfsnode 1.6.1 and update before you merge?
c89b7e3
to
8520e95
Compare
1.7.0 |
Improving a bit of test infra that I want to extract into go-unixfsnode so we can reuse it elsewhere (go-car, and here)