Skip to content
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

setup: Add __version__ attribute for the tasklib module #85

Merged
merged 1 commit into from
Sep 10, 2020

Conversation

tbabej
Copy link
Sponsor Member

@tbabej tbabej commented Sep 9, 2020

Providing __version__ attribute is a reasonably common convention among
packages in the Python ecosystem. Currently the only other reliable
alternative is to use pkg_resources.get_distribution method, however,
importing pkg_resources is notoriously slow [1,2].

Provide the __version__ attribute to provide an API interface to check
the version of tasklib at runtime.

Bump the version in order to reflect module API change.

[1] pypa/setuptools#510
[2] pypa/setuptools#926

Providing __version__ attribute is a reasonably common convention among
packages in the Python ecosystem. Currently the only other reliable
alternative is to use pkg_resources.get_distribution method, however,
importing pkg_resources is notoriously slow [1,2].

Provide the __version__ attribute to provide an API interface to check
the version of tasklib at runtime.

Bump the version in order to reflect module API change.

[1] pypa/setuptools#510
[2] pypa/setuptools#926
@tbabej
Copy link
Sponsor Member Author

tbabej commented Sep 9, 2020

@robgolding This would also need a PyPi release 🙏

@coveralls
Copy link

coveralls commented Sep 9, 2020

Coverage Status

Coverage increased (+0.01%) to 92.159% when pulling d035a34 on tbabej:develop into e0d5a3f on robgolding:develop.

@tbabej tbabej changed the title setup: Add __version__ attribute for the taskwiki module setup: Add __version__ attribute for the tasklib module Sep 9, 2020
@robgolding robgolding merged commit ef95cdf into GothenburgBitFactory:develop Sep 10, 2020
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.

None yet

3 participants