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

Multi package #23

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

Multi package #23

wants to merge 2 commits into from

Conversation

behrisch
Copy link
Contributor

allow running for multiple packages without re-entering credentials

@arcivanov
Copy link
Owner

Thanks for your PR. This needs to be thought through more thoroughly.
If batching is desired some packages may not be authorized for the user to delete while others will be.
The current behavior will be fail-on-first.

@behrisch
Copy link
Contributor Author

Yes, that was intended. There may be other things which go wrong (typo in package name etc.) so I wanted to stop as soon as the first error occurs to give immediate feedback and not to hide the error with subsequent success messages. Feel free to change it though :-)

@arcivanov
Copy link
Owner

So when I'm thinking "batch action" I'm thinking what git does when you delete multiple branches.

$ git branch -d a b c
error: branch 'a' not found.
error: branch 'b' not found.
error: branch 'c' not found.

Unless there is a rush, I'd like to think about it a bit.

@behrisch
Copy link
Contributor Author

no rush at all, I am working with my own branch and like your approach just as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants