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

Shrinking the size of the package release artifacts #122

Merged
merged 5 commits into from
Mar 29, 2022
Merged

Conversation

alxmrs
Copy link
Collaborator

@alxmrs alxmrs commented Mar 25, 2022

Fixes #121.

Before:

du -h dist/*
107M    dist/google-weather-tools-0.2.1.tar.gz
 88K    dist/google_weather_tools-0.2.1-py3-none-any.whl

After:

du -h dist/*
 44K    dist/google-weather-tools-0.2.2.dev2+g2b5c4da.d20220325.tar.gz
 88K    dist/google_weather_tools-0.2.2.dev2+g2b5c4da.d20220325-py3-none-any.whl

@alxmrs alxmrs requested review from pramodg and uhager March 25, 2022 18:59
@alxmrs
Copy link
Collaborator Author

alxmrs commented Mar 25, 2022

Note: the wheel file has stayed the same due to an issue spoken about here: https://stackoverflow.com/a/61274968

pypa/setuptools_scm#190

MANIFEST.in Outdated Show resolved Hide resolved
@alxmrs
Copy link
Collaborator Author

alxmrs commented Mar 28, 2022

After including config & documentation files, the dist folder looks like:

 du -h dist/*
 64K    dist/google-weather-tools-0.2.2.dev6+g9b3583e.tar.gz
 88K    dist/google_weather_tools-0.2.2.dev6+g9b3583e-py3-none-any.whl

@alxmrs alxmrs requested review from pramodg and fredzyda and removed request for uhager March 28, 2022 23:50
@alxmrs alxmrs merged commit c7a6c4f into main Mar 29, 2022
@alxmrs alxmrs deleted the smaller-tar branch March 29, 2022 00:02
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.

Shrink size of weather-tools tar to unblock upload to PyPi
2 participants