-
Notifications
You must be signed in to change notification settings - Fork 100
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
Provide make install to facilitate configuring nydus-snapshotter #24
Conversation
Codecov Report
@@ Coverage Diff @@
## main #24 +/- ##
==========================================
+ Coverage 34.37% 34.41% +0.03%
==========================================
Files 18 18
Lines 1277 1293 +16
==========================================
+ Hits 439 445 +6
- Misses 767 777 +10
Partials 71 71
Continue to review full report at Codecov.
|
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.
Add a "prefix: " to commit summary? e.g.
cmd: shorten default socket path
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.
Same to this patch, add "build" or "Makefile" prefix?
build: add install target to Makefile
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.
And we could use "misc: " as commit summary prefix in this patch
c31ec44
to
1b1c0be
Compare
Sure. All commits are prefixed now |
626fe08
to
2cbd433
Compare
2cbd433
to
0eab5ea
Compare
To shorten the default socket address and keep it consistent with the one in README.md Signed-off-by: Changwei Ge <[email protected]>
Thus to make it more convenient for users to install and setup nydus-snapshotter Signed-off-by: Changwei Ge <[email protected]>
Will be consumed when add/enable nydus-snapshotter service to systemd Signed-off-by: Changwei Ge <[email protected]>
Signed-off-by: Changwei Ge <[email protected]>
0eab5ea
to
72719d9
Compare
Users just perform
make install
and downloadnydusd
andnydus-image
before they can use nydus-snapshotter