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

Overhaul package-data sanity test #81427

Merged
merged 1 commit into from Aug 3, 2023

Conversation

mattclay
Copy link
Member

@mattclay mattclay commented Aug 3, 2023

SUMMARY

The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py.

ISSUE TYPE

Test Pull Request

The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py.
@ansibot ansibot added test This PR relates to tests. needs_triage Needs a first human triage before being processed. labels Aug 3, 2023
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Aug 3, 2023
@mattclay
Copy link
Member Author

mattclay commented Aug 3, 2023

The docs should be updated to reflect the new behavior of this sanity test.

@mattclay mattclay merged commit f894ce8 into ansible:devel Aug 3, 2023
78 checks passed
@mattclay mattclay deleted the package-data-overhaul branch August 3, 2023 20:24
mattclay added a commit to mattclay/ansible that referenced this pull request Aug 3, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py.
(cherry picked from commit f894ce8)

Co-authored-by: Matt Clay <matt@mystile.com>
mattclay added a commit to mattclay/ansible that referenced this pull request Aug 3, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py..
(cherry picked from commit f894ce8)

Co-authored-by: Matt Clay <matt@mystile.com>
mattclay added a commit to mattclay/ansible that referenced this pull request Aug 3, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py..
(cherry picked from commit f894ce8)

Co-authored-by: Matt Clay <matt@mystile.com>
sivel pushed a commit that referenced this pull request Aug 3, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py..
(cherry picked from commit f894ce8)
sivel pushed a commit that referenced this pull request Aug 3, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py.
(cherry picked from commit f894ce8)
mattclay added a commit that referenced this pull request Aug 4, 2023
The sanity test now only inspects the sdist and wheel instead of trying to install the sdist using setup.py..
(cherry picked from commit f894ce8)
@ansible ansible locked and limited conversation to collaborators Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
test This PR relates to tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants