Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Make --skip-previous-steps work with recursive flag! #33

Closed
madsnedergaard opened this issue May 14, 2020 · 1 comment · Fixed by #34
Closed

Make --skip-previous-steps work with recursive flag! #33

madsnedergaard opened this issue May 14, 2020 · 1 comment · Fixed by #34
Labels
bug Something isn't working

Comments

@madsnedergaard
Copy link
Member

Hey, I just realised that the --skip-previous-steps isn't working with recursive :(

So I guess we have to pass it down to the ctx.invoke() function, but I'm not quite sure what's the best approach for doing that - should I define a new "@click.argument()" for build, test, deploy functions?

Screenshot 2020-05-14 at 11 27 29

Any inputs on how to do this, @corradio?

@madsnedergaard madsnedergaard added the bug Something isn't working label May 14, 2020
@corradio
Copy link
Member

Another possibility is to set it globally, just like verbose:

image

It's definitely a hack though..

I tried to look at https://click.palletsprojects.com/en/7.x/api/#click.Context.invoke to see if one could make sure to preserve ctx.parent but couldn't see anything obvious.

@madsnedergaard madsnedergaard linked a pull request May 18, 2020 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants