stubtest: ignore name mismatches for positional only arguments

[srittau]

Currently, stubtest emits errors like this when the runtime name of a positional only name differs from the stub name:

error: sqlite3.dbapi2.register_converter is inconsistent, stub argument "__typename" differs from runtime argument "name"

In this particular case, the name of the argument changed between Python 3.10 and 3.11, requiring use to use a version check in the stubs.

I believe that name errors in positional only arguments should be ignored, as they have no runtime effect.

[hauntsaninja]

stubtest only complains about positional-only arguments if the names do not "approximately" match, where approximately is defined as

mypy/mypy/stubtest.py

Line 401 in 64a40e8

def names_approx_match(a: str, b: str) -> bool:

This has caught real issues (e.g. where an argument name change signals change in semantics), so I wouldn't like to throw this check away. Closing this as "as designed", but let me know if you feel strongly and I can reconsider.

[srittau]

I honestly dislike the false positives this causes when a positional only argument gets renamed. The extra version checks in typeshed are not only unnecessary, but cause extra busy work.

[sobolevn]

Plus one to @srittau, this is also very confusing to me.