PSVersionTable should have entry for OS and Platform - #3654
Conversation
|
James Truher (@JamesWTruher) - Please review this change as well |
There was a problem hiding this comment.
Shouldn't this use .ToString() as OSVersion is an object?
|
Steve Lee (@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()
There was a problem hiding this comment.
resolved
|
Steve Lee (@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.
|
Mike Richmond (@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