Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

WIP - Add README reference to Command Line Rosetta Stone#5929

Closed
mikemaccana wants to merge 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
mikemaccana:patch-1mikemaccana/PowerShell:patch-1Copy head branch name to clipboard
Closed

WIP - Add README reference to Command Line Rosetta Stone#5929
mikemaccana wants to merge 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
mikemaccana:patch-1mikemaccana/PowerShell:patch-1Copy head branch name to clipboard

Conversation

@mikemaccana

@mikemaccana mikemaccana commented Jan 17, 2018

Copy link
Copy Markdown

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
Copy Markdown
Contributor

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

@iSazonov

Copy link
Copy Markdown
Collaborator

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

@daxian-dbw

daxian-dbw commented Jan 18, 2018

Copy link
Copy Markdown
Member

@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
Copy Markdown
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

@TravisEz13 TravisEz13 left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
Collaborator

I suggest postponing this change until they become relevant.

@mikemaccana

mikemaccana commented Jan 19, 2018

Copy link
Copy Markdown
Author

@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
Copy Markdown
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
Copy Markdown
Member

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

@stale

stale Bot commented Mar 16, 2018

Copy link
Copy Markdown

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

stale Bot commented Mar 27, 2018

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Author

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

@daxian-dbw

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.