-
Notifications
You must be signed in to change notification settings - Fork 644
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
Consume NuGet.Services.Entities and NuGet.Services.Messaging.Email #6588
Conversation
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.
Tip: hide whitespace changes in diff settings :)
@@ -7,8 +7,6 @@ public static class CoreConstants | |||
{ | |||
public const string AdminRoleName = "Admins"; | |||
|
|||
public const int MaxPackageIdLength = 128; |
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 now a constant in NuGet.Services.Entities
.
@@ -1442,7 +1444,6 @@ | |||
<Compile Include="Services\IPackageService.cs" /> | |||
<Compile Include="Services\PackageService.cs" /> | |||
<Compile Include="Configuration\AppConfiguration.cs" /> | |||
<Compile Include="Constants.cs" /> |
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.
Renamed NuGetGallery.Constants
to GalleryConstants
to reduce/avoid type collisions in views, as both NuGetGalllery
and NuGet.Services.Entities
are known namespaces in each razor view.
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 PR:
ServerCommon.sln
NuGetGallery.Core
and consumesNuGet.Services.Messaging.Email
insteadHopefully, this will also help us limit the dependency on
NuGetGallery.Core
elsewhere :)