Skip to content
This repository has been archived by the owner on Mar 4, 2022. It is now read-only.

Add --no-bail option #69

Closed
wants to merge 7 commits into from
Closed

Add --no-bail option #69

wants to merge 7 commits into from

Conversation

exogen
Copy link
Contributor

@exogen exogen commented Dec 23, 2015

Adds an option to keep going even if one process ends with an error.

/cc @ryan-roemer

Fixes #64

Adds an option to keep going even if one process ends with an error.

/cc @ryan-roemer

Fixes #64
@@ -177,6 +178,7 @@ Flags:
* `--setup`: Single task to run for the entirety of `<action>`.
* `--queue`: Number of concurrent processes to run (default: unlimited - `0|null`)
* `--[no-]buffer`: Buffer output until process end (default: `false`)
* `--[no-]bail`: End all processes after the first failure (default: `true`)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a fan of this phrasing, but it's not coming to me.

@ryan-roemer
Copy link
Member

Looking good on first skim. I'll dive in more tomorrow. Thanks!

@ryan-roemer
Copy link
Member

Reviewed in pair with @exogen -- rebasing and merging!

@ryan-roemer
Copy link
Member

Released in builder@2.3.3

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

Successfully merging this pull request may close these issues.

None yet

2 participants