-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
Adopt consistent approach for defining InternalsVisibleTo in the repo #45748
Conversation
Thanks for your PR, @silamon. Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
Wow, that's a bigger change than I expected. Good job. Something's failing in DataProtection? |
...s/Kestrel/Transport.Sockets/src/Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.csproj
Outdated
Show resolved
Hide resolved
Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at. |
Yes, thanks for looking. Actually, it's not only in DataProtection. There are also a few cases in Kestrel. For example: There's a InternalsVisibleTo declared in Microsoft.AspNetCore.Server.Kestrel.Core for Microsoft.AspNetCore.Server.Kestrel. So, I'm not really sure why it remains inaccessible. The namespace of KestrelServerOptionsSetup does have "Internal" and most of those DataProtection classes also have that, but this is more of an assembly thing if I understand correctly. May it have something to do with the fact that I omitted the PublicKey? |
Nevermind, I think I saw the problem. |
Head branch was pushed to by a user without write access
The tests that use Moq and Castle.DynamicProxy are failing. |
Head branch was pushed to by a user without write access
Thanks |
Fixes #41244. This pull request moves all InternalsVisibleTo from AssemblyInfo.cs to their respective project file (.csproj). When reviewing, compare them 2-by-2 since the csproj file is always next to the AssemblyInfo.cs file.
One exception: the RateLimiting project had already the InternalsVisibleTo declared. In this project the AssemblyInfo.cs is removed without applying changes to .csproj project file.