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

Removing c9s workaround for pytest-container #2836

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

martinhoyer
Copy link
Collaborator

Pytest 6 incompatibility have been fixed in the 0.4.1 version of pytest-container.

Just removing the workaround, which installed upstream pytest on c9s.

@martinhoyer martinhoyer self-assigned this Apr 9, 2024
@martinhoyer martinhoyer added the code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. label Apr 9, 2024
@happz happz added this to the 1.33 milestone Apr 9, 2024
@happz happz added code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. test coverage Improvements or additions to test coverage of tmt itself labels Apr 9, 2024
@happz happz added the ci | full test Pull request is ready for the full test execution label Apr 9, 2024
@happz
Copy link
Collaborator

happz commented Apr 9, 2024

/packit test

@happz happz enabled auto-merge (squash) April 9, 2024 09:53
@psss psss merged commit c9f5f0e into teemtee:main Apr 10, 2024
18 of 20 checks passed
@psss psss assigned psss and unassigned martinhoyer Apr 10, 2024
@psss
Copy link
Collaborator

psss commented Apr 10, 2024

Failures were irrelevant, merged.

@lukaszachy lukaszachy modified the milestones: 1.33, 1.32.2 Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. test coverage Improvements or additions to test coverage of tmt itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants