Skip to content

Conversation

davidcaron
Copy link
Collaborator

Overview

This fixes pip install for the module. When running setup.py, the dependencies are not yet installed so the module can't be imported when we want to get the version number.

Related Issue / Discussion

bird-house/emu#67

Additional Information

https://packaging.python.org/guides/single-sourcing-package-version

Copy link
Member

@cehbrecht cehbrecht 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 :)

@cehbrecht
Copy link
Member

@davidcaron Thanks.

Do you think we should set author and email in setup.py from __version__.py?

The original cookiecutter didn't do this ... but it seems to work now and we can skip some redundancy.
I leave it to you to decide ... otherwise feel free to merge.

@cehbrecht cehbrecht added this to the 0.4.0 milestone Dec 4, 2018
@cehbrecht cehbrecht added the enhancement New feature or request label Dec 4, 2018
@cehbrecht cehbrecht merged commit 05c013a into bird-house:master Dec 4, 2018
@cehbrecht
Copy link
Member

@davidcaron done. Thanks :)

Zeitsperre pushed a commit that referenced this pull request Jan 5, 2021
* get version number without importing package

see:
https://packaging.python.org/guides/single-sourcing-package-version

* add note to __version__.py

* don't repeat author and email information
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants