Skip to content
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 a Dry-run option #236

Open
malobre opened this issue Aug 26, 2020 · 6 comments
Open

Add a Dry-run option #236

malobre opened this issue Aug 26, 2020 · 6 comments

Comments

@malobre
Copy link

malobre commented Aug 26, 2020

A dry-run option would be nice (e.g: ./install --dry-run).
It would allow testing new configs without wrecking your home.

@anishathalye
Copy link
Owner

That looks interesting, I'd be willing to add this to Dotbot. Were you interested in submitting a PR for this? Alternatively, I can implement it at some point.

A small note, this might behave in unintuitive ways for people who are using plugins that are not aware of --dry-run. Not sure how exactly we should handle this.

@malobre
Copy link
Author

malobre commented Aug 27, 2020

I could take a look, but my experience with python is close to zero.
And for the plugin problem, I haven't looked at the code yet so I don't know if it is possible, but my idea was to pass along the dry-run option to the plugins that do eventually support it and to otherwise not run the plugin at all (of course with a nice logging message).

@anishathalye
Copy link
Owner

Ah, maybe it makes sense for me to implement it, then. I like the idea of checking if plugins support it and skipping the plugin otherwise.

@joeyfigaro
Copy link

@anishathalye any progress or movement on this? Would like a dry run option as well. Just got a new machine and started setting up the basics before trying out dotbot.

@anishathalye
Copy link
Owner

It's on my todo list, but I haven't implemented this feature yet.

@jlumpe
Copy link

jlumpe commented Aug 4, 2022

@anishathalye Is this feature still on the to-do list? It would be very helpful to test your configuration.

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

No branches or pull requests

4 participants