-
Notifications
You must be signed in to change notification settings - Fork 232
fix deploy stage on travis #421
fix deploy stage on travis #421
Conversation
Codecov Report
@@ Coverage Diff @@
## master #421 +/- ##
=========================================
Coverage 82.81% 82.81%
Complexity 490 490
=========================================
Files 66 66
Lines 2019 2019
Branches 247 247
=========================================
Hits 1672 1672
Misses 265 265
Partials 82 82 Continue to review full report at Codecov.
|
The previous condition was always false, the deploy stage wasn't executed. Signed-off-by: Pavol Loffay <[email protected]>
863223a
to
e4f9948
Compare
I think wi will find out if it works once it's merged. |
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 guess there's only one way to check...
let's merge and hope 🙏 |
It looks promising https://travis-ci.org/jaegertracing/jaeger-client-java/builds/380130241 deploy stage is there |
There was a failure for deploy to the hub:
I will submit the following PR to fix it. |
Here is a comparison of deploy to maven
|
#389 changed travis configuration to use stages. There is a condition which enables
deploy
stage https://github.com/jaegertracing/jaeger-client-java/pull/389/files#diff-354f30a63fb0907d4ad57269548329e3R36. However it doesn't seem to work see https://travis-ci.org/jaegertracing/jaeger-client-java/builds/379736616deploy
is missingSigned-off-by: Pavol Loffay [email protected]