Skip to content

197 set up versioneer and test releasing #225

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

Merged
merged 6 commits into from
Mar 30, 2020
Merged

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
2 participants