Skip to content
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

Revert "Revert "Remove ref project from Microsoft.AspNetCore.Testing"" #2936

Merged

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Feb 4, 2020

Reverts #2882

We now know how to react to removing the Ref assembly for this project - see dotnet/aspnetcore#18720. We should resolve this for 3.1.3.

Part of #2784

@JunTaoLuo @dougbu PTAL

CC @Pilchie - marking as tell-mode since this is identical to #2817

@wtgodbe wtgodbe added servicing-consider Shiproom approval is required for the issue tell-mode labels Feb 4, 2020
@wtgodbe wtgodbe added this to the 3.1.3 milestone Feb 4, 2020
@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 5, 2020

Related: dotnet/aspnetcore#18803

@wtgodbe wtgodbe added area-infrastructure and removed servicing-consider Shiproom approval is required for the issue labels Feb 5, 2020
<!-- This package is internal, so we don't generate a package baseline. Always build against the latest dependencies. -->
<UseLatestPackageReferences>true</UseLatestPackageReferences>
<IsProjectReferenceProvider>false</IsProjectReferenceProvider>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI the need to disable $(IsProjectReferenceProvider) i.e. to do more than turn $(HasReferenceAssembly) off will go away once this repo picks up the ref/-related changes from dotnet/aspnetcore.

@wtgodbe wtgodbe merged commit 7a077d4 into release/3.1 Feb 13, 2020
@wtgodbe wtgodbe deleted the revert-2882-revert-2817-johluo/remove-testing-ref branch February 13, 2020 23:49
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Feb 28, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 2, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 25, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 27, 2020
@ghost ghost locked as resolved and limited conversation to collaborators May 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants