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

Set default value to false for Overwrite to prevent unnecessary warning #130

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

yorickbouma
Copy link
Contributor

@yorickbouma yorickbouma commented Mar 20, 2024


As all changes are public made available via PRs, you must agree to the terms of before making a contribution:

  • I represent that each of my contributions is entirely my original work
  • I have sole ownership of intellectual property rights to my contribution
  • I approve that the community is free to use my contribution

Make sure that you have checked all tick boxes before this PR can be approved.


At present, if you omit the deprecated input "Overwrite," it defaults to true and triggers a warning. The only way to prevent this currently is to explicitly set "Overwrite" to false, which seems a bit odd for a deprecated feature.

@yorickbouma
Copy link
Contributor Author

@liprec

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.

1 participant