corrected use of PSModulePath casing to be consistent with Windows PowerShell #3255
Conversation
| // and shared module paths. | ||
| string hklmModulePath = GetExpandedEnvironmentVariable("PSMODULEPATH", EnvironmentVariableTarget.Machine); | ||
| string hkcuModulePath = GetExpandedEnvironmentVariable("PSMODULEPATH", EnvironmentVariableTarget.User); | ||
| string hklmModulePath = GetExpandedEnvironmentVariable("PSModulePath", EnvironmentVariableTarget.Machine); |
There was a problem hiding this comment.
I count 8 references to this same string in code - there should be a const string that we use instead.
| @@ -1,3 +1,4 @@ | |||
| return | |||
There was a problem hiding this comment.
It seems we should remove.
There was a problem hiding this comment.
that was accidentally committed as those tests were failing due to external issue, will remove
| function Get-PassedArgsRoot { [Root]::new().passedIn } | ||
| $passedArgs = $args | ||
| class Root { $passedIn = $passedArgs } | ||
| function Get-PassedArgsRoot { [Root]::new().passedIn } |
There was a problem hiding this comment.
Didn't change anything here as PSModulePath isn't here. Not sure why this is showing up in the PR as a change.
There was a problem hiding this comment.
Don't see any whitespace difference either
make "PSModulePath" into const fixed some test workarounds due to failures for external reasons that wasn't meant to be checked in
| if (PropertyScope.CurrentUser == scope) | ||
| { | ||
| return ModuleIntrinsics.GetExpandedEnvironmentVariable("PSMODULEPATH", EnvironmentVariableTarget.User); | ||
| return ModuleIntrinsics.GetExpandedEnvironmentVariable("PSModulePath", EnvironmentVariableTarget.User); |
There was a problem hiding this comment.
Constants.PSModulePathEnvVar - right?
There was a problem hiding this comment.
Whitespace changes - add ?w=1 to see the url to see diffs ignoring whitespace.
There was a problem hiding this comment.
Did a search for "PSModulePath" and I think I got them all
make "PSModulePath" into const fixed some test workarounds due to failures for external reasons that wasn't meant to be checked in
|
@TravisEz13 merge? |
5 participants
addresses #3227