Adding -Extension and -LeafBase switches to Split-Path#2721
Adding -Extension and -LeafBase switches to Split-Path#2721daxian-dbw merged 3 commits intoPowerShell:masterPowerShell/PowerShell:masterfrom powercode:split-path-switchespowercode/PowerShell:split-path-switchesCopy head branch name to clipboard
Conversation
|
Hi @powercode, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! The agreement was validated by Microsoft and real humans are currently evaluating your PR. TTYL, MSBOT; |
fd24ea2 to
e8e7f40
Compare
There was a problem hiding this comment.
should there be explicit tests for things like:
"$level2_1Full$fileExt$fileExt"
to ensure that the trimming is not over-zealous?
e8e7f40 to
ef0abc9
Compare
There was a problem hiding this comment.
What's the difference between LeafSet and LeafBaseSet?
There was a problem hiding this comment.
The comment seems to be duplicated.
There was a problem hiding this comment.
The default value of _parent used to be true. The default for a switch is False. Please revisit.
There was a problem hiding this comment.
Good catch. Fixing and rebasing.
…No functionallity changes Using auto properties when no when there is no logic in getter/setter Removing unused code Removing redundant qualifiers Removing Redundant initializers
Extension and LeafBase are specializations of Leaf, and uses System.IO.Path.GetExtension and System.IO.Path.GetFilenameWithoutExtension to extract parts from the Leaf
ef0abc9 to
33e266e
Compare
|
@charub your comments have been addressed, can you please take another look? If the change looks good, please approve the PR. |
Fixes #2700
Extension and LeafBase are specializations of Leaf, and uses System.IO.Path.GetExtension
and System.IO.Path.GetFilenameWithoutExtension to extract parts from the Leaf
Tests added in test\powershell\Modules\Microsoft.PowerShell.Management\FileSystem.Tests.ps1