-
Notifications
You must be signed in to change notification settings - Fork 816
Compile using ref assemblies #2483
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
src/Logging/Logging.Testing/src/Microsoft.Extensions.Logging.Testing.csproj
Outdated
Show resolved
Hide resolved
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.
LGTM other than the few comments
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.
I'll approve after the biggest change is categorized
@@ -0,0 +1,7 @@ | |||
<Project> |
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.
This is fine but only if it is not creating a new (useless) package
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.
It is being produced. Will remove.
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.
@dougbu I marked out the two changes that are made for compiling against ref. Addressing Will's feedback mostly involved adding comments and removing errors that are irrelevant in Extensions so I'm not marking them out. The rest of the changes are just copied over from AspNetCore.
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.
Looks good.
Please confirm no new Microsoft.Extensions.ProviderAliasUtilities.Sources package is created.
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework\n\nCommit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework\n\nCommit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework Commit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework Commit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework Commit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework Commit migrated from dotnet/extensions@e771c42
* Compile using ref assemblies * Use updated ResolveReferences.target from AspNetCore * Generate ref assemblies for .NET framework Commit migrated from dotnet/extensions@e771c42
https://github.com/aspnet/AspNetCore-Internal/issues/1773