Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I checked the results of
python -m build
after merging #159 and found that both the .tar.gz and .whl outputs were about 10 MB. This was because the entire /doc/ directory—including many images added by that other PR—was included in both files.This PR adjusts our setuptools config so that the /doc/ directory is:
This follows the practice of
coverage
and several other packages I looked at. It also ensures that users who install from .whl (with pip) don't wind up with an extra directory namedsite-packages/doc/
that contains the message-ix-models docs. (AFAIK, distributing these 'dangling' directories outside the ones strictly related to the package is bad practice.)With these changes, the .whl file is also nice and slim at under 600 kB.
How to review
Note that the CI checks all pass.
PR checklist
Add or expand tests;coverage checks both ✅Add, expand, or update documentation.N/A, packaging changes onlyUpdate doc/whatsnew.