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

Expose __version__ module-level attribute #217

Merged
merged 2 commits into from
Oct 24, 2019

Conversation

Elkasitu
Copy link

@Elkasitu Elkasitu commented Nov 9, 2018

It is possible for integrators to need to check the flanker version
being run by the client in order to enable/disable flanker
functionality depending on which python version is being run.

Indeed, if the python version is 2.x, any version of flanker should
work, however if the version is 3.x, only flanker versions >= 0.9 are
compatible.

This patch makes flanker comply with PEP396 [1] and allows for runtime
version checking in cases where a hard dependency requiring flanker >=
0.9 is not possible (e.g. optional e-mail validation)

[1] https://www.python.org/dev/peps/pep-0396/#specification

It is possible for integrators to need to check the flanker version
being run by the client in order to enable/disable flanker
functionality depending on which python version is being run.

Indeed, if the python version is 2.x, any version of flanker should
work, however if the version is 3.x, only flanker versions >= 0.9 are
compatible.

This patch makes flanker comply with PEP396 [1] and allows for runtime
version checking in cases where a hard dependency requiring flanker >=
0.9 is not possible (e.g. optional e-mail validation)

[1] https://www.python.org/dev/peps/pep-0396/#specification
@mailgun-ci
Copy link

Can one of the admins verify this patch?

@thrawn01 thrawn01 merged commit 65fe5c1 into mailgun:master Oct 24, 2019
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.

3 participants