-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add an option to allow manifest-path to be used #7
base: master
Are you sure you want to change the base?
Conversation
Switch to use manifest-path for cargo multi
I see that you've dropped the output indentation. Was it on purpose ? If so, could you elaborate why please ? |
I did not purposely drop the indentation (as I did not realize that you had indentation), although the way that you had previously implemented it meant that it buffered all of the output before dumping stdout at the end, which means that for a long running cargo command there is no feedback on progress and it looks like multi has hung. |
Workaround for imp#9
Pin to clap 2.20
Add workspace support
Not the manifest paths themselves...
Produce of manifest _directories_
Here is my updated PR which makes the manifest-path optional. I have also switched to output stdout/stderr, rather than hiding it.