-
Notifications
You must be signed in to change notification settings - Fork 304
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
use setuptools find_packages, exclude tests, docs and examples from dist #368
Conversation
oh, and we may need to add a temporary pin for |
Codecov Report
@@ Coverage Diff @@
## master #368 +/- ##
==========================================
+ Coverage 69.29% 69.30% +0.01%
==========================================
Files 56 56
Lines 6135 6135
Branches 804 804
==========================================
+ Hits 4251 4252 +1
Misses 1616 1616
+ Partials 268 267 -1
Continue to review full report at Codecov.
|
Cool (i guess) looks like jedi's not breaking anything... |
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.
LGTM and makes sense - thanks Nick. I'd prefer that someone more familiar with packaging/distribution than myself to review this as well prior to merging though.
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.
Thanks!
* update locks to jupyterlab 3.4.1 * print pip version
use setuptools find_packages, exclude tests, docs and examples from dist
This:
find_packages
fromsetupbase.py
setuptools.find_packages
toexclude
things that might be python-module-likeIn
site-packages/
,tests/
,docs/
, etc. are dumping grounds of all the packages that either don't use this flag. In this state, it's not even worth distributing them, as there's basically zero chance they'll work. Putting them in-tree, on the other hand, is very nice, as downstreams can ensure that their packages work, which helps catch issues sooner, but that's for another day...