Thomas Grainger

I'd be ok with merging this without 8d3dbb8 if all side effecting functions are also forbidden. ``` requests.post(url, data) # side effecting magic ```

six... | old | new -- | -- | -- UserDict | UserDict.UserDict | collections.UserDict UserList | UserList.UserList | collections.UserList UserString | UserString.UserString | collections.UserString

looks like they were commented out by @brettcannon in https://github.com/python-modernize/python-modernize/commit/39816c9a2e5faf885de7dcc29e7a95226566a4cd#diff-3fdceef3e184860491d74b794f086590R58-R60

@fake-name it may be better to use https://github.com/davidhalter/jedi

@sobolevn a PR would be great thank you!

@jayclassless thanks I made a PR https://github.com/jayclassless/setoptconf/pull/3

@jayclassless nice, can you also upload a wheel so nobody will be impacted by future setuptools changes?

> A place where the current checker does something weird is requiring a comma after "kwargs" which is definitely not going to see more arguments after it in most code....

there's since been a `six.StringIO`

Imho this would need support in six with a `@six.python_2_bool_compatible` decorator. Otherwise it should be added to fissix without the backwards compatible use of six