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

Replace .Sources package references with local code #4801

Merged
merged 10 commits into from
Dec 14, 2018

Conversation

natemcmaster
Copy link
Contributor

This copies code from aspnet/Extensions into this repo. This code is not actually used in aspnet/Extensions, so it seems more appropriate to put the shared sources here.

Having the code locally makes it easier to

  • debug and develop this code
  • source link
  • ensure patches and servicing are done correctly

@natemcmaster
Copy link
Contributor Author

FYI @ajaybhargavb @JunTaoLuo - this will conflict with the .csproj file changes you're making as a part of #4246

@natemcmaster
Copy link
Contributor Author

/AzurePipelines run AspNetCore-ci

@azure-pipelines
Copy link

Success! 2 created and queued.

@natemcmaster natemcmaster merged commit 13ae005 into dotnet:release/2.1 Dec 14, 2018
@natemcmaster natemcmaster deleted the sources branch December 14, 2018 16:36
dougbu pushed a commit to dougbu/razor-compiler that referenced this pull request Nov 17, 2021
…e#4801)

Replaces package references the following packages with local code:

Microsoft.AspNetCore.Certificates.Generation.Sources
Microsoft.Extensions.CopyOnWriteDictionary.Sources 
Microsoft.Extensions.ClosedGenericMatcher.Sources
Microsoft.Extensions.ObjectMethodExecutor.Sources
Microsoft.Extensions.PropertyActivator.Sources
Microsoft.Extensions.PropertyHelper.Sources
Microsoft.Extensions.RazorViews.Sources
Microsoft.Extensions.SecurityHelper.Sources
Microsoft.Extensions.StackTrace.Sources
Microsoft.Extensions.WebEncoders.Sources

Commit migrated from dotnet/aspnetcore@13ae0057fbb1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant