Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove plain #1700

Merged
merged 1 commit into from
Nov 14, 2021
Merged

remove plain #1700

merged 1 commit into from
Nov 14, 2021

Conversation

MagnetarRocket
Copy link
Contributor

@MagnetarRocket MagnetarRocket commented Nov 13, 2021

remove "plain" as documentation does not show it, and is a invalid settings option in winget, removal as it's a non-functional option and would benefit confused text editor users to match the options shown in the documentation and available settings in the actual winget.

This proposal can be canceled if the implementation of "plain" is planed further down the road.


"Are you working against an issue?" No issue has been filed for this, in fact the fix might be a little bit premature as it was filed before a issue.

Microsoft Reviewers: Open in CodeFlow

remove "plain" as documentation does not show it, and is a invalid settings option in winget, removal as it's a non-functional option and would benefit confused text editor users to match the options shown in the documentation and available settings in the actual winget.

This proposal can be canceled if the implementation of "plain" is planed further down the road.
@MagnetarRocket MagnetarRocket requested a review from a team as a code owner November 13, 2021 08:39
@yao-msft yao-msft merged commit 9b66b2d into microsoft:master Nov 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants