microsoft / microsoft/winget-cli
[New Feature]: Enable-WingetSetting and Disable-WingetSetting should return confirmation
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 26.4k
- Forks
- 1.8k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 15
Description
Description of the new feature/enhancement
Currently, running Enable-WingetSetting LocalManifestFiles or Disable-WingetSetting LocalManifestFiles provides no output and it is not clear whether the command succeeded or not. The native commands winget settings --<Enable|Disable> LocalManifestFiles provide output that the setting was either enabled or disabled, It would provide much better feedback to the user if the PowerShell module returned the setting and its state as a confirmation that it was set
Proposed technical implementation details (optional)
PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> Enable-WingetSetting LocalManifestFiles
Name Value
---- -----
LocalManifestFiles True
PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs>
PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> Disable-WingetSetting LocalManifestFiles
Name Value
---- -----
LocalManifestFiles False
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the implementations of Enable-WingetSetting and Disable-WingetSetting, then trace how each command currently handles its setting change. The work is done when both commands report the setting name and its resulting true or false state as shown in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- powershell
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100