Skip to content

use versioneer#84

Merged
emiliom merged 1 commit intoODM2:masterfrom
ocefpaf:versioneer
Jun 13, 2017
Merged

use versioneer#84
emiliom merged 1 commit intoODM2:masterfrom
ocefpaf:versioneer

Conversation

@ocefpaf
Copy link
Copy Markdown
Member

@ocefpaf ocefpaf commented Jun 8, 2017

With versioneer all we need to do is to tag a new release and the version should be properly propagated everywhere. Note that I am using a pep440 style, that means tags like v0.6.0-alpha will be translated to 0.6.0a0.

There are also some advantages, like GitHub hashes and/or a dirty flag when creating releases from branches or from uncommitted changes. This improves reproducibility and manageability of releases.

Closes #83

PS: There is a lot of boilerplate code in this PR. but this is done only once. Most of this code won't need any modification in the future.

@ocefpaf ocefpaf requested a review from sreeder June 8, 2017 16:41
@ocefpaf ocefpaf mentioned this pull request Jun 8, 2017
@emiliom
Copy link
Copy Markdown
Member

emiliom commented Jun 12, 2017

@sreeder, do you want to review this PR, or should I go ahead and merge? FYI, @ocefpaf also recently implemented versioneer in wofpy, ODM2/WOFpy#138 (already merged)

Comment thread setup.cfg
versionfile_build = odm2api/_version.py
tag_prefix = v
parentdir_prefix =

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that most of this PR is boilerplate. What needs reviweing is in this file.
Here we are using the prefix v for the version number in the tags, like v1.0.0. A tag like 1.0.0 would be ignored by this configuration! And we are also conforming to the PEP440 for version identification and dependency specification. If you agree with those than it should be OK to merge.

Copy link
Copy Markdown
Contributor

@sreeder sreeder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. It will be nice to have this ease of use upgrade, thanks for working on it @ocefpaf!

@emiliom emiliom merged commit 48f6dca into ODM2:master Jun 13, 2017
@ocefpaf ocefpaf deleted the versioneer branch June 13, 2017 16:55
@ocefpaf ocefpaf mentioned this pull request Jun 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants