-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Moving towards a whitelist approach for MANIFEST.in #8109
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8109 +/- ##
=========================================
+ Coverage 65.89% 65.9% +<.01%
=========================================
Files 485 485
Lines 22917 22922 +5
Branches 2537 2537
=========================================
+ Hits 15102 15106 +4
- Misses 7683 7684 +1
Partials 132 132
Continue to review full report at Codecov.
|
I think this line needs to be changed to use Running |
There's a lot of stuff in
assets
that doesn't need to be shipped and/or served with Superset. The current setup ofMANIFEST.in
starts out by doing an initialrecursive-include
againststatic
and then selectively excludes several items. As a result, several files are being left behind, such as terser caches, .rc files, package.json, etc.Fixes #8107
CATEGORY
Choose one
REVIEWERS
@mistercrunch