Replace use of whitelist with allowlist/clearlist and blacklist with denylist/blocklist
hvongsachang opened this issue · comments
hvongsachang commented
In a similar fashion to this issue: rails/rails#33677, it would be nice if vulture could replace use of whitelist with allowlist/clearlist and blacklist with denylist/blocklist for better terminology. If not replace, then perhaps add support to detect allowlist
and blocklist
.
Jendrik Seipp commented
I think we can even find a more specific name for the modules that fake code usage. I like the name "fake module" and would be happy to review a pull request that changes
- "whitelist" to "fake module" in text,
- "whitelist" to "fake_module" in code,
- and "--make-whitelist" to "--make-fake-module".
SvenWarnke commented
I am not sure if --make-fake-module is as intuitive.