-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Feature request: simulate / dry-run option #1488
Comments
This is a good think but a little bit tricky. What to return from test for example? if (test('...')) {
run('a');
} else {
run('b');
} |
I see your point. Theoretically you would like to see what would be run, based on if the test succeeded or not. So in case of:
it would be awesome to see something like:
and for
you would see:
But I don't know if this is easily implemented 🙄 |
I think the thing we miss the most is an option to dump/print the order of executing of the tasks. So when we do something like
So we have one flow-overview of all the different tasks and the order in which they will be executed |
are you aware of the |
Yes, but that actually does the deploy. And I want to be able to see/debug my deploy without actually performing the deploy. And imo this should be debuggable / viewable using the CLI ^^ |
Ok, I'm almost finished with a DebugCommand ^^ |
This is kinda done in master! Checkout --plan option! |
It would be nice to be able to dry-run a deploy when setting up the different tasks.
Especially to see the different sub-tasks that are run. I now have to look up the order of the deploy-tasks in the documentation.
And also when setting up the deploy configuration, I actually have to run it with
-vvv
to see the different commands that are executed.It would be nice to just see the output written to the screen when you have the option
--dry-run
added or--simulate
or somethingI understand that would mean all the recepices or core functions need to be aware of this.
But I think it would help tremendously when debugging / setting up a deploy.
And I would be happy to help with getting this done.
The text was updated successfully, but these errors were encountered: