-
Notifications
You must be signed in to change notification settings - Fork 10.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
Make security project ref #4618
Conversation
0a02108
to
da3ba41
Compare
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.
A good start. I added a few comments about little things that can change in the project file, and I pushed a commit to your PR to trim some stale content from the README in src/Security
One thing you should for sure revert is the removal of the git submodules. :) Still required for now.
cc @Tratcher - you may be interested in viewing the source code layout.
src/Mvc/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Microsoft.AspNetCore.Mvc.ViewFeatures.csproj
Outdated
Show resolved
Hide resolved
...est/Microsoft.AspNetCore.Mvc.FunctionalTests/Microsoft.AspNetCore.Mvc.FunctionalTests.csproj
Outdated
Show resolved
Hide resolved
src/Security/Authentication.Cookies/src/Microsoft.AspNetCore.Authentication.Cookies.csproj
Outdated
Show resolved
Hide resolved
src/Security/Authentication.JwtBearer/src/Microsoft.AspNetCore.Authentication.JwtBearer.csproj
Outdated
Show resolved
Hide resolved
src/Security/Authentication.WsFederation/samples/WsFedSample/WsFedSample.csproj
Outdated
Show resolved
Hide resolved
src/Security/Authentication/src/Microsoft.AspNetCore.Authentication.csproj
Outdated
Show resolved
Hide resolved
src/Security/Authorization.Policy/src/Microsoft.AspNetCore.Authorization.Policy.csproj
Outdated
Show resolved
Hide resolved
src/Security/CookiePolicy/test/Microsoft.AspNetCore.CookiePolicy.Test.csproj
Outdated
Show resolved
Hide resolved
The layout's a little odd, especially for the AuthN samples and tests that pull in stuff from other folders. How about:
Also, the src folder can be flattened when it's the only folder for a given package. E.g. |
I'll take over this PR since I need it for Identity. |
Updated after talking to @natemcmaster
And don't worry about flattening the solo src dirs, it sounds like each of these will get paired with a ref dir. |
99bd440
to
069f80d
Compare
6e19bdd
to
94f75f5
Compare
Updated |
No description provided.