-
Notifications
You must be signed in to change notification settings - Fork 918
(#533) Support packages.config file in upgrade #1132
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
base: master
Are you sure you want to change the base?
(#533) Support packages.config file in upgrade #1132
Conversation
Nice! |
Your commit summary is missing some words ;) |
37a13e2
to
3046bc8
Compare
Oops - is that better? |
Much better. 👍 |
Please don't merge master - Merges are very bad in feature branches. I thought we had that outlined in our CONTRIBUTING document, but it's more implied than called out directly - https://github.com/chocolatey/choco/blob/master/CONTRIBUTING.md#submit-pull-request-pr and https://github.com/chocolatey/choco/blob/master/CONTRIBUTING.md#respond-to-feedback-on-pull-request |
Sorry, I did that via the github web interface which I assumed would do the right thing. I will fix it up.... |
dbe8ecf
to
879778b
Compare
Implements a subset of the functionality, support for packages.config files in the upgrade command. Also extended the tests to test the new code. I did not see any value in implementing package.config support for uninstall because the problem I am interested in solving is being able to upgrade multiple packages each with a specific version in one command. Since uninstall (generally?) doesn't require specifying a version this can be done on the uninstall command line, i.e. just give the package list to uninstall.
879778b
to
4ef0a40
Compare
@Apteryx0 @ferventcoder what's the timeline for getting this merged in? I'm facing a similar use case currently where this feature would make my life easier. |
This has a chance to get into the next version. The linked ticket #533 has the milestone. Currently prioritized in 0.10.9. |
Russell Mora seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Implements a subset of the functionality, support for packages.config
files in the upgrade command. Also extended the tests to test the new
code. I did not see any value in implementing package.config support for
uninstall because the problem I am interested in solving is being able
to upgrade multiple packages each with a specific version in one
command. Since uninstall (generally?) doesn't require specifying a
version this can be done on the uninstall command line, i.e. just give
the package list to uninstall.
Closes #533