PSVersionTable should have entry for OS and Platform#3654
PSVersionTable should have entry for OS and Platform#3654mirichmo merged 1 commit intoPowerShell:masterfrom chunqingchen:bugfix4
Conversation
|
@JamesWTruher - Please review this change as well |
There was a problem hiding this comment.
Shouldn't this use .ToString() as OSVersion is an object?
|
@SteveL-MSFT Thank you, your comment has been resolved. |
There was a problem hiding this comment.
Part of me thinks we shouldn't make any changes for FullCLR, but if we're making the change it seems this line would fail as the right hand side should use ToString()
|
@SteveL-MSFT comment resolved. |
There was a problem hiding this comment.
Please fix the indentation within this file. It looks like you may have added tab characters instead of spaces. My comment applies to all changes made within this file.
|
@mirichmo I have tried to modify all the tab previous file contains into space. |
|
Thanks. I'll merge it once it passes CI. I restarted since Travis failed with a possible unrelated issue. |
This is to resolve #1635
$PSVersionTable.OS now outputs [Runtime.InteropServices.RuntimeInformation]::OSDescription ([System.Environment]::OSVersion if not under coreclr)
$PSVersionTable.Platform outputs [System.Environment]::OSVersion.Platform