WebCmdlets relocate to Common Path#19122
WebCmdlets relocate to Common Path#19122CarloToso wants to merge 2 commits intoPowerShell:masterfrom
Conversation
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
|
@CarloToso I don't agree to move files because this break commit history. We merged so many PRs in last months that it is critical to keep the history. |
|
You are right, keeping the commit history is more is more important than reordering |
|
@CarloToso Invoke-WebRequest is still partial class in one file. You could merge the parts. |
PR Summary
If this PR is good, merge soon, I'd like to continue working on the affected files
PR Context
Contributes to #5610, #5662 by markekraus
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).