Skip to content

Conversation

@mikemaccana
Copy link

@mikemaccana mikemaccana commented Jan 17, 2018

PR Summary

A modern version of the traditional Unix Rosetta Stone, which is maintained, includes Powershell, and has more detail than the table below.

PR Checklist

Note: Please mark anything not applicable to this PR NA.

A modern version of the traditional Unix Rosetta Stone, which is maintained, includes Powershell, and has more detail than the table below.
TravisEz13
TravisEz13 previously approved these changes Jan 18, 2018
@alexandair
Copy link
Contributor

@TravisEz13 @joeyaiello Most of PowerShell commands in Command Line Rosetta Stone are not applicable to PowerShell Core.

@iSazonov
Copy link
Collaborator

I can not open the site in the latest IE version. 😕

@daxian-dbw
Copy link
Member

daxian-dbw commented Jan 18, 2018

@TravisEz13 As @alexandair mentioned, some commands there don't work with PowerShell Core, like Get-WmiObject and *-EventLog commands, as well as [Environment]::OSVersion, which doesn't give the correct OS version.

@TravisEz13
Copy link
Member

I looked at CommandLine Rosetta stone... you can make changes, but I didn't look and see if you could add a new option.

@TravisEz13 TravisEz13 dismissed their stale review January 19, 2018 01:02

see comments

Copy link
Member

@TravisEz13 TravisEz13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the commands are correct for PowerShell core.

@iSazonov
Copy link
Collaborator

I suggest postponing this change until they become relevant.

@mikemaccana
Copy link
Author

mikemaccana commented Jan 19, 2018

@alexandair 'Most of' seems a little... surprising. Thanks @daxian-dbw for being more specific, I can update the document for the Powershell Core equivalents later today (or you can if you get there first).

@TravisEz13 Just edit https://github.com/certsimple/rosetta-stone/blob/master/rosetta-stone.md - the entire document is built from that.

@TravisEz13
Copy link
Member

Thanks, if you are going to do the edits. Ping here and hopefully a few people can verify.

@TravisEz13 TravisEz13 changed the title Add README reference to Command Line Rosetta Stone WIP - Add README reference to Command Line Rosetta Stone Feb 8, 2018
@TravisEz13
Copy link
Member

Please remove WIP from title when the feedback has been addressed.

@stale
Copy link

stale bot commented Mar 16, 2018

This PR has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed if no further activity occurs within 10 days. Thank you for your contributions.

@stale stale bot added the Stale label Mar 16, 2018
@stale
Copy link

stale bot commented Mar 27, 2018

This PR has be automatically close because it is stale. If you wish to continue working on the PR, please first update the PR, then reopen it.
Thanks again for your contribution.
Community members are welcome to grab these works.

@stale stale bot closed this Mar 27, 2018
@mikemaccana
Copy link
Author

Chasing this up, sorry about the delay but it's been hard finding docs on a few of the items. Summarising:

Thanks.

@mikemaccana
Copy link
Author

@TravisEz13 if you could let me know what you think re: above I can make any/all necessary changes.

@daxian-dbw
Copy link
Member

Test-Connection should be there in 6 but still seems to be missing, despite #1146. Maybe waiting on a new release. Does anyone know?

The original Test-Connection implementation was removed in #5277 to make way for @iSazonov's new implementation (PR #5328). However, the new Test-Connection didn't make in 6.0.

@mikemaccana
Copy link
Author

Thanks @daxian-dbw I'll leave Test-Connection in so it's updated when 6.1 is out. Do you (or @TravisEz13 ) know about Resolve-DnsName per my comment above?

@TravisEz13
Copy link
Member

@mikemaccana Resolve-DnsName is missing, at least on non-windows platforms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants