Skip to content

[Feature] Remove HelpSystem old formatting code - #5907

Merged
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
iSazonov:helpsystem-remove-old-code
Jan 17, 2018
Merged

[Feature] Remove HelpSystem old formatting code#5907
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
iSazonov:helpsystem-remove-old-code

Conversation

@iSazonov

@iSazonov Ilya (iSazonov) commented Jan 15, 2018

Copy link
Copy Markdown
Collaborator

PR Summary

Remove HelpSystem old formatting code for CommandInfo. This code was intended for the PowerShell 2.0 version which deprecated currently.
Also now we use new HTTPS HelpUri links, and old HTTP HelpUri links for PS 2.0 may no longer work.

PR Checklist

@lzybkr Jason Shirk (lzybkr) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice catch, I removed some of those from the format ps1xml, but missed this one.

I see a couple other places that can be removed:

System.Management.Automation\engine\hostifaces\HostUtilities.cs
946:            if ($PSVersionTable.PSVersion -lt ([version] '3.0'))
963:            if ($PSVersionTable.PSVersion -lt ([version] '3.0'))

@iSazonov

Copy link
Copy Markdown
Collaborator Author

CI Travis platform has a problem - I restarted jobs.

@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 5674bf3 into PowerShell:master Jan 17, 2018
@iSazonov
Ilya (iSazonov) deleted the helpsystem-remove-old-code branch January 17, 2018 17:52
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WG-Interactive-HelpSystem help infrastructure and formatting of help

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants