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

(doc) choco install update packages.config params #1085

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Stunkymonkey
Copy link

Description Of Changes

some parameters are not documented. taken from https://github.com/chocolatey/choco/blob/develop/src/chocolatey/infrastructure.app/configuration/PackagesConfigFilePackageSetting.cs

Motivation and Context

Users should not have to read code.

Testing

  • I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.
    • Issue #

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

N/A

@pauby
Copy link
Member

pauby commented Oct 16, 2024

Can you fix the PR so only the changes show? If you are making whitespace changes, please use a separate commit.

@pauby pauby added the 0 - Waiting on User Insufficient information for issue or PR, issue may be closed if no response from user label Oct 17, 2024
@Stunkymonkey Stunkymonkey force-pushed the add-missing-packages.config-param branch from 80af8a9 to a230cce Compare October 17, 2024 19:49
@Stunkymonkey
Copy link
Author

@pauby sure. is this better? (this was automatically applied editor-config)

@Stunkymonkey Stunkymonkey force-pushed the add-missing-packages.config-param branch from a230cce to e36ed1d Compare October 22, 2024 18:44
@Stunkymonkey
Copy link
Author

ping @pauby

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Waiting on User Insufficient information for issue or PR, issue may be closed if no response from user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants