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

Simplify numbering of versions #468

Merged
merged 1 commit into from Dec 9, 2016

Conversation

Projects
None yet
2 participants
@Godin
Member

Godin commented Dec 9, 2016

JARs of releases that we we publish into Maven central repository should have major, minor and patch components in version, but not qualifier, i.e. simply 0.7.8. Qualifier should remain in MANIFEST files.

Initial discussion: https://groups.google.com/d/msg/jacoco-dev/HNWPzop4ZI4/pvmfWsSSCAAJ

@Godin Godin added this to the 0.7.8 milestone Dec 6, 2016

@Godin Godin self-assigned this Dec 6, 2016

@Godin Godin changed the title from Simplify version numbering to Simplify numbering of versions Dec 9, 2016

@Godin Godin requested a review from marchof Dec 9, 2016

@marchof

This comment has been minimized.

Show comment
Hide comment
@marchof

marchof Dec 9, 2016

Member

@Godin Beside the change log, is there anything else we need to modify for this? Or will this happen as part of the release process?

Member

marchof commented Dec 9, 2016

@Godin Beside the change log, is there anything else we need to modify for this? Or will this happen as part of the release process?

@Godin

This comment has been minimized.

Show comment
Hide comment
@Godin

Godin Dec 9, 2016

Member

@marchof so far I believe that this affects only release procedure

Member

Godin commented Dec 9, 2016

@marchof so far I believe that this affects only release procedure

@marchof

marchof approved these changes Dec 9, 2016

@marchof

This comment has been minimized.

Show comment
Hide comment
@marchof

marchof Dec 9, 2016

Member

@Godin Please feel free to merge after checks are green.

Member

marchof commented Dec 9, 2016

@Godin Please feel free to merge after checks are green.

@Godin Godin merged commit bb8e46c into master Dec 9, 2016

4 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@Godin Godin deleted the issue-468 branch Dec 9, 2016

@jacoco jacoco locked and limited conversation to collaborators Jan 11, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.