Skip to content

Add support for NO_OVERWRITE config option#97

Open
Chaz6 wants to merge 1 commit intoamietn:masterfrom
Chaz6:patch-1
Open

Add support for NO_OVERWRITE config option#97
Chaz6 wants to merge 1 commit intoamietn:masterfrom
Chaz6:patch-1

Conversation

@Chaz6
Copy link

@Chaz6 Chaz6 commented Aug 9, 2021

No description provided.

@amietn
Copy link
Owner

amietn commented Aug 14, 2021

Thank you for contributing!

I believe this does not work if no_overwrite is set to False in the config file. It would still be interpreted as True.

One solution would be to call config.getboolean() instead of config.get() for options of boolean type.

@Chaz6
Copy link
Author

Chaz6 commented Aug 15, 2021

Thank you for the feedback, I will amend the patch.

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

Successfully merging this pull request may close these issues.

2 participants