-
Notifications
You must be signed in to change notification settings - Fork 224
Conversation
- Rename related classes - Rename related functional tests
Just a note : https://github.com/aspnet/MusicStore/blob/dev/test/E2ETests/Common/DeploymentUtility.cs#L313 needs to change as well with this change. |
{ | ||
c.Description = "Build NuGet packages for the project in given directory"; | ||
c.Description = "Pack NuGet packages for the project in given directory"; |
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.
Build is still a better verb to use.
Changed script names in the latest commit: We also have |
@@ -142,7 +142,7 @@ public int Main(string[] args) | |||
}); | |||
}); | |||
|
|||
app.Command("build", c => | |||
app.Command("pack", c => | |||
{ | |||
c.Description = "Build NuGet packages for the project in given directory"; |
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.
? "Produce NuGet packages..."
from my perspective but I did not check thoroughly enough to be the primary reviewer. will this lead to additional changes in the tooling schema? other (new) downstream changes? |
Need a rebase after #1117 is checked in. |
We will do this in release branch. Please check the new PR: #1124 |
parent #967
Follow-up reaction: #1035
Impacted units: Tooling, test scripts, build scripts, wiki pages