Skip to content

Conversation

parleur
Copy link
Contributor

@parleur parleur commented Dec 14, 2015

Fusion between #860 & #862. Test the version number correctness in unit test, normalize it when exec setup.py

closes #842

minrk and others added 3 commits December 14, 2015 11:47
avoids mismatch between wheel/sdist upload on PyPI due to setuptools normalization of prerelease tags.
test directly in the version number file, as the version number is
likely to be committed before running tests, so we want **really quick
failure**.
@Carreau Carreau added this to the 4.1 milestone Dec 14, 2015
Copy link
Member

Choose a reason for hiding this comment

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

version_info should be (4, 1, 0, 'b1')

@minrk minrk changed the title fusion between #860 & #862 Use PEP 440 prerelease tags Dec 14, 2015
@Carreau
Copy link
Member

Carreau commented Dec 14, 2015

I took care of a few of the lasts comments.

Copy link
Member

Choose a reason for hiding this comment

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

from this line down can be removed

Copy link
Member

Choose a reason for hiding this comment

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

Oops. I took care of it too.

@minrk
Copy link
Member

minrk commented Dec 15, 2015

One more note about lingering unused code in _version.py, but otherwise +1.

@Carreau
Copy link
Member

Carreau commented Dec 15, 2015

One more note about lingering unused code in _version.py, but otherwise +1.

Thanks for your patience !

minrk added a commit that referenced this pull request Dec 15, 2015
Use PEP 440 prerelease tags
@minrk minrk merged commit 92fa310 into jupyter:master Dec 15, 2015
@minrk
Copy link
Member

minrk commented Dec 15, 2015

Thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

beta sdist filename with extra dot.

3 participants