-
Notifications
You must be signed in to change notification settings - Fork 717
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
migrate to zipkin-java'like deployment #161
Comments
@adriancole I'm all for it. It is good to have consistency and it prevents I'm the only one who can make releases for brave. It the release process described somewhere? |
I've extracted the maven sync plugin which @abesto and I worked on. Once this is in, we can switch brave to use openzipkin process openzipkin-attic/centralsync-maven-plugin#1 |
added a bintray package and requested it added to jcenter https://bintray.com/openzipkin/maven/brave/view |
so.. for releasing brave from OZ. We've the parts to do this now. |
This re-uses the deployment flow from zipkin-java which has the following features: * Makes Brave releaseable by anyone with access to the Bintray org * Snapshot publishing to http://oss.jfrog.org/artifactory/oss-snapshot-local * Fully automated release to maven central via release-N.N.N tag * Releases also published to https://jcenter.bintray.com Notably, this migrates the maven group id to `io.zipkin.brave`. This needs to be communicated on release! Fixes #161
here's the change to turn this on! #178 |
http://oss.jfrog.org/artifactory/oss-snapshot-local/io/zipkin/brave/ has snapshots now |
pushed tag release-3.8.0 and seeing if it works! |
Nice! 👍 |
Awesome sauce. 🎆 🍰 |
Most impactful would be changing the group id from "com.github.kristofa" to "io.zipkin.brave". If we do, then we can use bintray instead of sonatype. This has the advantage of group access controls and nice things like auto-signing of jars. Most importantly, we can use the same process to release zipkin-java and brave.
The text was updated successfully, but these errors were encountered: