Skip to content

docs: Update installation instructions for WinGet - #6716

Merged
mislav merged 3 commits into
cli:trunkfrom
vegerot:winget-instructions
Dec 12, 2022
Merged

docs: Update installation instructions for WinGet#6716
mislav merged 3 commits into
cli:trunkfrom
vegerot:winget-instructions

Conversation

@vegerot

@vegerot vegerot commented Dec 10, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

After running winget install --id GitHub.cli I spent fifteen minutes trying to figure out why it didn't work. My first thought was something to do with the PATH, so I opened a new PowerShell tab. When that didn't work, I spent 14 more minutes trying to figure out what went wrong.

I think giving specific instructions will help other developers like me.

Continuation from #4772 (comment)

@vegerot
vegerot requested a review from a team as a code owner December 10, 2022 23:49
@vegerot
vegerot requested review from mislav and removed request for a team December 10, 2022 23:49
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Dec 10, 2022
@cliAutomation

cliAutomation commented Dec 10, 2022

Copy link
Copy Markdown
Contributor

Thanks for the pull request! We're a small team and it's helpful to have context around community submissions in order to review them appropriately. Our automation has closed this pull request since it does not have an adequate description. Please edit the body of this pull request to describe what this does, then reopen it.

vegerot
vegerot commented Dec 10, 2022
View reviewed changes

@vegerot vegerot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Continuation from #4772 (comment)

After running winget install --id GitHub.cli I spent fifteen minutes trying to figure out why it didn't work. My first thought was something to do with the PATH, so I opened a new PowerShell tab. When that didn't work, I spent 14 more minutes trying to figure out what went wrong.

I think giving specific instructions will help other developers like me.

@mislav mislav reopened this Dec 12, 2022
mislav
mislav approved these changes Dec 12, 2022
View reviewed changes

@mislav mislav left a comment •
edited
Loading

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.

Thank you!

Ref. microsoft/winget-cli#222

@mislav
mislav enabled auto-merge (squash) December 12, 2022 13:30
@mislav
mislav merged commit f57249f into cli:trunk Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@mislav mislav mislav approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vegerot @cliAutomation @mislav