chocolatey / chocolatey/choco

Update the wording of the except option on the Upgrade command

Open
#3,147 0 comments 0 reactions 0 assignees View on GitHub
0 - Triaging Enhancement
Dominant language
C#
Stars
11.5k
Forks
960
PR merge metrics
No merged PRs in 30d

Description

### Checklist

- [X] I have verified this is the correct repository for opening this issue.
- [X] I have verified no other issues exist related to my request.

### Is Your Feature Request Related To A Problem? Please describe.

_No response_

### Describe The Solution. Why is it needed?

The description of the `except` option is a little ambiguous about where it applies, and if packages listed could be upgraded even if included on the except option.

https://github.com/chocolatey/choco/blob/370e36632670d3e3d25a874ea8175551891bca50/src/chocolatey/infrastructure.app/commands/ChocolateyUpgradeCommand.cs#L168-L171

### Additional Context

When doing `choco upgrade all --except package-a`, it's somewhat implied that `package-a` will not be upgraded, but if there is a dependency resolution that requires `package-a` to be upgraded, it will in fact be upgraded.

### Related Issues

_No response_

Contributor guide

Open the contributing guide

Research direction

Read src/chocolatey/infrastructure.app/commands/ChocolateyUpgradeCommand.cs at lines 168-171 and inspect how the Upgrade command describes the --except option. Clarify whether excluded packages can still be upgraded through dependency resolution, using the issue's `choco upgrade all --except package-a` example as the completion check.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
cli
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.