Skip to content

Conversation

@wyli
Copy link
Contributor

@wyli wyli commented Mar 28, 2020

Fixes #197.

Description

  • set up versioneer for monai,
    so that python -c 'import monai; print(monai.__version__)' returns a PEP440 versioning
  • merged several codebase configs into setup.cfg
  • adds a section of the release procedure in contributing.md

Testing

To test the correctness, I've tagged a commit of this branch with version number 0.1a1.dev1:

  • running python -c 'import monai; print(monai.__version__)'
    returned the correct version number. this was tested from multiple cases:
  • adding new commits on top of the tag,
    the version number becomes 0.1a1.dev1+1.ge6de7ac. Which is expected. the number indicates 1 revision ahead of 0.1a1.dev1 with hash e6de7ac.

Things not covered

the CONTRIBUTING.md now covers the building/releasing procedure,
automating the procedure will need another iteration

Status

READY

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New tests added to cover the changes
  • Docstrings/Documentation updated

@wyli wyli requested review from IsaacYangSLA and Nic-Ma March 28, 2020 17:55
@wyli wyli changed the title WIP 197 set up versioneer and test releasing 197 set up versioneer and test releasing Mar 28, 2020
@wyli wyli force-pushed the 197-set-up-versioneer branch 3 times, most recently from 0bca1e8 to f0ad0d5 Compare March 29, 2020 16:45
        - tests for python 3.x
        - adds license for setup.py
        - adds python3.x required in setup.cfg
@wyli wyli force-pushed the 197-set-up-versioneer branch from f0ad0d5 to 4087626 Compare March 29, 2020 17:02
Copy link
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

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

Looks good to me now.
Thanks.

@Nic-Ma Nic-Ma merged commit ec1d73d into master Mar 30, 2020
@wyli wyli deleted the 197-set-up-versioneer branch May 21, 2020 13:30
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.

Release workflow

3 participants