-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix a set of test issues caused by an old blob feed and try to fix up the nullable change we rushed in #45420
Conversation
|
It could be due to a temporary issue with the network or storage account. Now, these tests have suddenly passed. Therefore, I will close this PR. |
Reactivating as this has templating test fixes and nullable fixes as well. Additionally, I think we want the change to remove the blob. It has risk of keeping it around and removing simplifies things. Only one test failed which I'll talk to Daniel about. Microsoft.NET.Publish.Tests.GivenThatWeWantToStoreAProjectWithDependencies.store_nativeonlyassets failed |
… have easy access to
Fixes #45417
Changes included