Repository navigation
Disable PackageManagementTests - #2703
Aditya Patwardhan (adityapatwardhan) wants to merge 1 commit into
Conversation
Package management tests disabled temporarily to unblock daily builds.
|
Jianyun (@jianyunt) Dongbo Wang (@daxian-dbw) Can you have a look? |
|
I think it's better to use a file-level try-finally to disable them. |
|
Aditya Patwardhan (@adityapatwardhan) If we are just temporarily disabling those tests, wouldn't it be much easier to just |
|
Dongbo Wang (@daxian-dbw) that way we get reporting about skipped / pending tests and it's useful. |
|
sergei (@vors) Yes, but we are removing them from PS/PS, so it's doesn't matter that we don't keep track of skipping/pending for them. BTW, I chatted with Aditya Patwardhan (@adityapatwardhan) and learnt that Jianyun (@jianyunt) asked him to directly delete those tests from PS/PS, which is even better. |
|
"Waiting on author" then? |
|
Tests are deleted in the different PR. |
Package management tests disabled temporarily to unblock daily builds.