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

[stable-2.15] Clean up MANIFEST.in #81414

Merged
merged 2 commits into from Aug 2, 2023

Conversation

mattclay
Copy link
Member

@mattclay mattclay commented Aug 2, 2023

SUMMARY
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

MANIFEST.in

* Remove unnecessary MANIFEST.in commands

This eliminates more setuptools warnings during build.

* Sort MANIFEST.in

Now that only include commands are used, the order of the commands no longer effects the build output.

(cherry picked from commit 6e325d9)
* Omit `hacking` directory from `MANIFEST.in`

* Update package-data sanity test

(cherry picked from commit b93a628)
@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. backport This PR does not target the devel branch. labels Aug 2, 2023
@mattclay mattclay merged commit 5265794 into ansible:stable-2.15 Aug 2, 2023
83 checks passed
@mattclay mattclay deleted the manifest-cleanup-2.15 branch August 2, 2023 20:30
@sivel sivel removed the needs_triage Needs a first human triage before being processed. label Aug 3, 2023
@ansible ansible locked and limited conversation to collaborators Aug 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport This PR does not target the devel branch. bug This issue/PR relates to a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants