FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update README and delete docs that were migrated by sdwheeler · Pull Request #1790 · PowerShell/PSScriptAnalyzer · GitHub

/ PSScriptAnalyzer Public

Update README and delete docs that were migrated - #1790

Merged
James Truher (JamesWTruher) merged 2 commits into
PowerShell:masterfrom
sdwheeler:sdw-i1946982
May 23, 2022
ConversationCommits (2)ChecksFiles changed
Merged

Update README and delete docs that were migrated#1790
James Truher (JamesWTruher) merged 2 commits into
PowerShell:masterfrom
sdwheeler:sdw-i1946982

Conversation

Sean Wheeler (sdwheeler) commented May 9, 2022 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Fixes #1723 - Update README and delete docs that were migrated

Steven Bucher (StevenBucher98) approved these changes May 9, 2022
View reviewed changes

Steven Bucher (StevenBucher98) left a comment •
edited
Loading

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Edit: sorry just found few last things when doing another pass through

Steven Bucher (StevenBucher98) reviewed May 9, 2022
View reviewed changes
Comment thread README.md Outdated
Apply suggestions from code review
6bc2804 
Co-authored-by: Steven Bucher <stevenbucher8@gmail.com>
Steven Bucher (StevenBucher98) reviewed May 9, 2022
View reviewed changes
Comment thread README.md

- Windows PowerShell 3.0 or greater
- PowerShell Core 7.0.3 or greater on Windows/Linux/macOS
- Docker (tested only using Docker Desktop on Windows 10 1809)

Steven Bucher (StevenBucher98) May 9, 2022

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I cant seem to find thedocker information in the doc pages now? Is it outdated and thus why it was removed?

Sean Wheeler (sdwheeler) May 9, 2022

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

The supported platforms information is in the Overview article in docs. All of the discussion of the docker containers was for the supported versions of PowerShell. That information is in the PowerShell setup docs and doesn't really belong here. See https://docs.microsoft.com/powershell/scripting/install/powershell-in-docker

James Truher (JamesWTruher) merged commit 1e2351f into PowerShell:master May 23, 2022
Sean Wheeler (sdwheeler) deleted the sdw-i1946982 branch December 7, 2022 14:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Steven Bucher (StevenBucher98) StevenBucher98 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Move conceptual documentation to the docs repo

3 participants

Back | FazBrowse Home | New Git URL