Skip to content

MANIFEST.in: Add test.py to sdist - #74

Merged
un33k merged 1 commit into
un33k:masterfrom
jayvdb:patch-1
Mar 25, 2019
Merged

un33k merged 1 commit into
un33k:masterfrom
jayvdb:patch-1

Conversation

@jayvdb

@jayvdb jayvdb commented Mar 12, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.079% when pulling be2ea48 on jayvdb:patch-1 into b8be7d6 on un33k:master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 92.079% when pulling be2ea48 on jayvdb:patch-1 into b8be7d6 on un33k:master.

@un33k

un33k commented Mar 23, 2019

Copy link
Copy Markdown
Owner

Usure why you want the test file in the final package?

@un33k un33k closed this Mar 23, 2019
@jayvdb

jayvdb commented Mar 24, 2019

Copy link
Copy Markdown
Contributor Author

See my explanation at thumbor/thumbor#1179 .

The sdist is the source distribution.

It is the wheel which shouldnt include the extra unnecessary stuff.

@un33k un33k reopened this Mar 25, 2019
@un33k
un33k merged commit ee1741b into un33k:master Mar 25, 2019
@jayvdb

jayvdb commented Mar 26, 2019

Copy link
Copy Markdown
Contributor Author

Thanks.

@jayvdb

jayvdb commented Mar 26, 2019

Copy link
Copy Markdown
Contributor Author

Packaging update submitted https://build.opensuse.org/request/show/688522

@un33k

un33k commented Sep 8, 2026

Copy link
Copy Markdown
Owner

This is Dojo, posting a maintainer-authorized follow-up linking this PR to #191.

Explicit source-test inclusion and source archive checks are provided for newly built distributions. Historical published archives are not modified.

The default algorithm remains legacy; improved output rules are opt-in. This note does not announce a published release, and no individual PR is being merged by this follow-up. Thank you for the contribution and discussion.

🚀 Generated with Dojo ⛩️

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.

3 participants