Force keyword arguments

[xLaszlo]

This might be addressed elsewhere, but how do I force keyword arguments or reformat the code to add them automatically?

This is what I found for Flake:

https://pypi.org/project/flake8-force-keyword-arguments/

I'd like to see the same behaviour, maybe with automatically adding them at fixing.

[charliermarsh]

I believe this plugin only checks for functions defined in the same module as the usage. Is that correct? (I tested it too, and it seems to be the case.) We could support it, but it seems quite limited.

[cleder]

It does check for functions outside the same module, e.g:

pygeoif/factories.py:46:27: FKA100 re.compile's call uses 2 positional arguments, use keyword arguments.
pygeoif/functions.py:150:25: FKA100 zip_longest's call uses 2 positional arguments, use keyword arguments.
pygeoif/functions.py:158:20: FKA100 math.isclose's call uses 2 positional arguments, use keyword arguments.
...

An autofix would be nice, even if that would be limited to the same module as the usage

[xLaszlo]

Yes, I think it checks other modules as well. (Thanks for dealing with this, especially this fast!)

I used the following command to run it for a project and then increased the argument and checked the relevant mistakes.

flake8 NewVersion/ --select FKA1 --kwargs-max-positional-arguments 0

After that, I ditched it because I didn't want to flake in the project just.

I am not sure what is the general policy on this. It probably would be great to enforce it or project files, but not for built-in or common commands. There are a lot of those errors that @cleder has shown.

[cleder]

• re.compile can be re.compile(pattern= ..., flags=...) ✔️
• itertools.zip_longest accepts positional arguments only for the values to be zipped, and forces keyword arguments for fillvalue ✖️
• math.isclose would be math.isclose(a=.., b=...) which does not improve readability ✖️

I do not think it is worth the effort at this time