Improve node info for invalid-name

[cdce8p]

Description

At the moment invalid-name for a function argument is emitted on the function node. That does work, however it limits where pylint: disable can be set. Consider the following example:

class SomeClass:
    def func(
        self,
        var,
        invalidName,
    ):
        pass

Only invalidName is wrong. With this change it's now possible to add pylint: disable directly behind it instead of after def func.

This appears to be fully backwards compatible. I.e. both with disable invalid-name:

# old
class SomeClass:
    def func(  # pylint: disable=invalid-name
        self,
        var,
        invalidName,
    ):
        pass

# new
class SomeClass:
    def func(
        self,
        var,
        invalidName,  # pylint: disable=invalid-name
    ):
        pass

[cdce8p]

I would like to target 2.11.2 for this one if thats ok.

[coveralls]

Pull Request Test Coverage Report for Build 1308371750

• 3 of 4 (75.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 93.114%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pylint/checkers/base.py	3	4	75.0%

Totals
Change from base Build 1307547189:	0.0%
Covered Lines:	13400
Relevant Lines:	14391

---

💛 - Coveralls

[Pierre-Sassoulas]

I would like to target 2.11.2 for this one if that's ok.

This does not feel like a "fix" but so does all the typing we already added in 2.11.2. I feel like we have already a lot in 2.11.2 and could practically release 2.12 instead at this point.

[cdce8p]

This does not feel like a "fix" but so does all the typing we already added in 2.11.2. I feel like we have already a lot in 2.11.2 and could practically release 2.12 instead at this point.

That might indeed be the better solution 😅
Technically it is a new feature, but since it's fully backwards compatible it could be ok.