-
Notifications
You must be signed in to change notification settings - Fork 886
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
Rewrite the chocolateysetup.psm1
module
#3255
Labels
Comments
vexx32
added a commit
to vexx32/choco
that referenced
this issue
Jul 6, 2023
This script has barely been touched in the last decade, in part because it's difficult to follow. Gave it a much needed revision to remove unnecessary clutter, make its code patterns more consistent, and be much more sensible in how it declares and calls commands, so that we can be much more confident that things are working how we expect them to. This should help make it much easier to make changes to going forward, should we ever need to.
vexx32
added a commit
to vexx32/choco
that referenced
this issue
Jul 6, 2023
This script has barely been touched in the last decade, in part because it's difficult to follow. Gave it a much needed revision to remove unnecessary clutter, make its code patterns more consistent, and be much more sensible in how it declares and calls commands, so that we can be much more confident that things are working how we expect them to. This should help make it much easier to make changes to going forward, should we ever need to.
10 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist
Is Your Feature Request Related To A Problem? Please describe.
The
chocolateysetup.psm1
module that thechocolatey
nupkg loads to install Chocolatey CLI itself was first written a very long time ago, and it uses code style that is not maintainable. As a result, we tend to avoid touching it as much as possible.Describe The Solution. Why is it needed?
Rewrite the module using more modern PowerShell patterns and make it much easier to maintain and make any necessary changes to going forward.
Additional Context
No response
Related Issues
No response
The text was updated successfully, but these errors were encountered: