-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update pytest
to 8.4.1
#4016
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
Update pytest
to 8.4.1
#4016
Conversation
Let me know if I can merge it |
This can be merged as soon as CI passes if there are no concerns from your side. It will allow me to prepare a PR to OpenVINO. Thanks! |
pip-conflicts failed |
It seems this PR and openvinotoolkit/openvino#31025 need to be merged at the same time since they're codependent and conflict with each other. Let's not do this on Friday as there's always a risk of concurrent merging not going as planned. |
Today is Friday. Do you have the rights to merge? If not, I can grant it. Feel free to merge |
I misread your message. I thought you proposed merging on Friday. OK. Up to you |
Agreed with @Wovchena to force merge this with failing pip conflicts job. The check fails due to After merging this PR I'll prepare a PR to OpenVINO which should fix the broken check in OMZ, since it will realign the pytest versions. |
### Details: - Instead of #31025, which can't be rebased - openvinotoolkit/open_model_zoo#4016 has been merged, which should fix the pip conflicts check ### Tickets: - N/A Signed-off-by: p-wysocki <[email protected]>
### Details: - Instead of openvinotoolkit#31025, which can't be rebased - openvinotoolkit/open_model_zoo#4016 has been merged, which should fix the pip conflicts check ### Tickets: - N/A Signed-off-by: p-wysocki <[email protected]>
Update
pytest
to fix a CI check in OpenVINO in openvinotoolkit/openvino#31025