-
Notifications
You must be signed in to change notification settings - Fork 387
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
chore: migrate to setup.cfg #666
Conversation
Codecov Report
@@ Coverage Diff @@
## master #666 +/- ##
==========================================
- Coverage 94.47% 94.45% -0.02%
==========================================
Files 57 57
Lines 8375 8368 -7
==========================================
- Hits 7912 7904 -8
- Misses 463 464 +1
Continue to review full report at Codecov.
|
Hey @ceache Really sorry for not having check this before :( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Programming Language :: Python | ||
Programming Language :: Python :: 3 | ||
Programming Language :: Python :: 3.7 | ||
Programming Language :: Python :: 3.8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is Python 3.9 missing?
Wasn't in scope originally but I'll add it in a next PR
…On Tue, Sep 20, 2022, 12:53 Stephen Sorriaux ***@***.***> wrote:
***@***.**** approved this pull request.
Thank you!
------------------------------
In setup.cfg
<#666 (comment)>:
> +description = "Higher Level Zookeeper Client"
+long_description = file: README.md, CHANGES.md
+long_description_content_type = text/markdown
+license = Apache 2.0
+license_file = LICENSE
+platform = any
+keywords = zookeeper, lock, leader, configuration
+classifiers =
+ Development Status :: 5 - Production/Stable
+ License :: OSI Approved :: Apache Software License
+ Intended Audience :: Developers
+ Operating System :: OS Independent
+ Programming Language :: Python
+ Programming Language :: Python :: 3
+ Programming Language :: Python :: 3.7
+ Programming Language :: Python :: 3.8
is Python 3.9 missing?
—
Reply to this email directly, view it on GitHub
<#666 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIFTHWG4BZHKRH6VZ3UG7LV7HTX5ANCNFSM5V6EOUAA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
closes #644 #662 #663
Why is this needed?
Migrate to python packaging best practices.
Proposed Changes
Does this PR introduce any breaking change?
No