-
Notifications
You must be signed in to change notification settings - Fork 393
Disable tests on official build #9450
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
Conversation
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.
The assumption being that tests ran during PR?
|
@joeloff Good call. Edit: Oh wait. No, Windows is always ran and Linux/Mac are PR only. Okay, I gotta condition it then for Windows. I'll do that too. |
…s a separate build condition for that situation.
|
Build succeeded. Merging. |
|
@backport to release/8.0.1xx |
|
We're still showing up on the dashboard I think from the servicing branches it looks like. requested copilot backport as the backport bot appears broken. |
Co-authored-by: marcpopMSFT <[email protected]>
…ase-8-0-1xx Backport PR #9450: Disable tests on official builds
Fixes: #9429
Summary
In the official pipeline,
-integrationTestwas specified. However, the build would run both integration tests and unit tests. It seems the easiest way was to specify/p:Test=falseon these builds, which disabled running tests altogether.Build Run
No tests running: https://dnceng.visualstudio.com/internal/_build/results?buildId=2817954&view=results