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

Specifying a whitelist should make blacklist default to everything. #41

Open
analog-cbarber opened this issue Jul 7, 2021 · 1 comment

Comments

@analog-cbarber
Copy link

If you have specified a whitelist, then an empty blacklist doesn't really make any sense.

Instead if there is a whitelist perhaps the blacklist should default to everything if not explicitly
specified.

To make such a change backward compatible, we could add additional names to use for
blacklist/whitelist (probably a good idea anyway) and only apply the new behavior to the
new names. E.g. 'exclude' for 'blacklist' and 'include', 'require' and/or 'depends' for the whitelist.

@xhochy
Copy link
Member

xhochy commented Jul 7, 2021

👍 This could be done with regards to what is already open at #11

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

2 participants