-
Notifications
You must be signed in to change notification settings - Fork 9
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
Proposal: additive configure
#66
Comments
There are two ways I see this going: 1. Always replace a configured option.
would give you
2. Lists are additive
would give you
Not sure which kind of behavior is more intuitive and easy-to-use. As for the old behavior of replacing a file entirely, we could always tie that to the |
Update: I no longer think there is a need for this functionality. I just directly edit the config file like other dotfiles. I think we just need to prompt user to optionally edit the config file themselves when existing file is detected. Maybe add documentation for the syntax. |
Description
In the
clean
doc: https://nbautoexport.drivendata.org/cleaning/I tried to add a folder to exclude, and it overwrote my old non-default
export_formats
config.I think it would be more natural to not have everything overwritten.
What I Did
The text was updated successfully, but these errors were encountered: