immutables.Map() as default arg should not trigger B008

[hukkin]

flake8-bugbear currently allows using the immutable frozenset() and tuple() types as default args of a function.

Would it be possible to also whitelist immutables.Map from https://github.com/MagicStack/immutables ?

[hukkin]

To elaborate, would be great if the code below would not give me B008 error:

import immutables
from immutables import Map


def this_should_be_fine(mapping = immutables.Map()):
    pass

def this_also(mapping = Map()):
    pass

[cooperlees]

I'm going to pass this one to our core maintainer @ambv since this immutables.Map is from his MagicStack ...

I personally don't like us using not stdlib objects, but I'm open to this being an extra install maybe? If we find immytables.Map importable, don't fire the line?

[hukkin]

I personally don't like us using not stdlib objects

Yeah I can see how this could be a bit of a bad precedent and a scope creep. Still would love having this one exception, as the stdlib does not include an immutable mapping/dict type, and using immutables types (tuple, frozenset, immutables.Map) as default arg makes me not have to write this convention a million times

def func(mapping = None):
    if mapping is None:
        mapping = {}

Thanks for considering!

[escaped]

How about making the "immutable function calls" configurable via the ini files. Something like pep8-naming did for classmethod/staticmethod. It's not documented, but I can add exceptions to the config file like

[flake8]
# pep8-naming
classmethod-decorators =
  classmethod,  # buildin
  validator,  # pydantic
  root_validator,  # pydantic

[hukkin]

I no longer have a need for this now that #144 is merged.

[hukkin]

Closing this now that we have #204 and #144