Skip to content
This repository was archived by the owner on Jan 28, 2022. It is now read-only.

Conversation

tristan0x
Copy link
Member

@tristan0x tristan0x commented Aug 2, 2017

Do not assume version of setuptools installed and explicitly install packaging.

Fix #4

@coveralls
Copy link

coveralls commented Aug 2, 2017

Coverage Status

Coverage remained the same at 89.777% when pulling d15f6e8 on fix/version_class into d68a48f on master.

@tristan0x tristan0x merged commit 55106ce into master Aug 2, 2017
Copy link
Contributor

@ferdonline ferdonline left a comment

Choose a reason for hiding this comment

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

If the only thing you need from "packaging" is the ability of read the version from GIT, give a look at setuptools_scm. It is more recent and has got a lot of attention.

@tristan0x
Copy link
Member Author

tristan0x commented Aug 2, 2017 via email

@ferdonline
Copy link
Contributor

Actually I see now that you import "hpcbench" to extract the version.
I used to do that too, until I read this: https://packaging.python.org/guides/single-sourcing-package-version/
Anyway I would be interested in knowing what the Version object does.
Cheers

@tristan0x
Copy link
Member Author

tristan0x commented Aug 3, 2017 via email

@tristan0x tristan0x deleted the fix/version_class branch August 7, 2017 06:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants