-
-
Notifications
You must be signed in to change notification settings - Fork 525
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
compat: Hatchling 1.26 #7526
compat: Hatchling 1.26 #7526
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7526 +/- ##
==========================================
- Coverage 82.38% 82.37% -0.01%
==========================================
Files 339 339
Lines 50995 50995
==========================================
- Hits 42010 42008 -2
- Misses 8985 8987 +2 ☔ View full report in Codecov by Sentry. |
c5f4316
to
1342c64
Compare
1342c64
to
d5937bf
Compare
eb44b56
to
7127eb2
Compare
|
||
from pathlib import Path | ||
|
||
EXPECTED_SIZES_MB = { |
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.
What is the current size of these artifacts?
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.
I added a print statement to make this somewhat easy to get. If needed, I can output it to the Github summary.
conda file size: 21.65 MB
npm file size: 18.32 MB
sdist file size: 28.31 MB
whl file size: 26.41 MB
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.
@philippjfr when you're back, you can adjust the limits set and merged in this PR in case you think they're too high.
Current sizes:
conda file size: 21.65 MB
npm file size: 18.32 MB
sdist file size: 28.31 MB
whl file size: 26.41 MB
Limits implemented:
{
"conda": 25,
"npm": 25,
"sdist": 30,
"whl": 30,
}
Changes made in Hatchling 1.26.0 no longer ignore
node_modules
. I think it is related to pypa/hatch#1643.List of differences in file list between this PR and 1.5.4: diff.txt