-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[release/6.0] Backport deletion of playwright tests #45852
Conversation
Hi @HaoK. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document. |
Per discussion with @captainsafia and @TanayParikh these playwright tests were removed in 7.0 so this just backports their removal to 6.0 so we avoid any extra playwright issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/azp help |
Supported commands
See additional documentation. |
/azp list |
CI/CD Pipelines for this repository: |
/azp run aspnetcore-ci |
Azure Pipelines successfully started running 1 pipeline(s). |
@wtgodbe this PR is green, is there any other process we need to follow to get this in, or is this sufficient for the build team to merge during the window? |
This PR is test-only. Any reason not to merge it @HaoK❔ |
Nope, should be good to merge! |
These tests were removed in 5247048 for 7.0, and due to pw having some infrastructure costs, better to remove these in 6.0 rather than update pw infrastructure