melos icon indicating copy to clipboard operation
melos copied to clipboard

request: add --fail-fast param to bootstrap

Open cedvdb opened this issue 2 years ago • 3 comments
trafficstars

Is there an existing feature request for this?

  • [X] I have searched the existing issues.

Command

melos bs --fail-fast

Description

fail-fast on bootstrap would ensure pub get is not run on the totality of the project before finding the failed result.

Reasoning

takes additional ci minutes for failed attempts

Additional context and comments

No response

cedvdb avatar Jan 03 '23 17:01 cedvdb

That makes sense. Maybe we should even default to fail fast when we detect that we are running in CI. We can still allow --fail-fast and --no-fail-fast to override the default behavior.

blaugold avatar Jan 03 '23 21:01 blaugold

default when we detect that we are running in CI

Is there a reason not to always do it ? My issue is specifically aimed at CI, but it annoyed me outside CI too

cedvdb avatar Jan 03 '23 22:01 cedvdb

If I'm running melos bootstrap locally, I think I would prefer that as many packages get bootstrapped as possible, so I know which packages exactly have issues.

blaugold avatar Jan 04 '23 18:01 blaugold