-
Notifications
You must be signed in to change notification settings - Fork 3
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
Issue 21: maven artifacts publish #22
base: feature-separate-adapters
Are you sure you want to change the base?
Issue 21: maven artifacts publish #22
Conversation
Build steps for docker and dockerPush. Dockerfiles with jar addition
Signed-off-by: Ravi Rajamani <ravi.rajamani@emc.com>
Signed-off-by: Ravi Rajamani <ravi.rajamani@emc.com>
Codecov Report
@@ Coverage Diff @@
## feature-separate-adapters #22 +/- ##
============================================================
Coverage 72.77% 72.77%
Complexity 95 95
============================================================
Files 10 10
Lines 518 518
Branches 47 47
============================================================
Hits 377 377
Misses 102 102
Partials 39 39 Continue to review full report at Codecov.
|
Signed-off-by: Ravi Rajamani <ravi.rajamani@emc.com>
7f5fd97
to
630bf9f
Compare
@@ -0,0 +1,104 @@ | |||
/** |
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.
The steps in this file is the same as for other pravega repositories. No changes have been made other naming.
71a6d7d
to
b18f5c7
Compare
Change log description
The purpose of this change is to publish maven artifacts from pravega-storage-adapters repository - one each for the storage adapters in this repository.
Purpose of the change
Fixes: #21
What the code does
It uses a more recent plugin by name maven-publish rather than the older maven plugin which gives us a cleaner syntax and semantics.
How to verify it
These changes are verified using the gradle command './gradle publishToMavenLocal'