napoleon: Don't use attribute type annotation for constructor parameters

[hoodmane]

Fix a bug where if a class has an attribute and a constructor parameter with the same name, napoleon will use the type annotation from the attribute for the constructor argument.

Feature or Bugfix

• Bugfix

Detail

Suppose we have a class that looks like

class Example:
    """
    A Class

    Parameters
    ----------
    blah:
        Description of parameter blah
    """
    def __init__(self, blah: int):
        pass

    blah: bool

The processed docstring looks like:

A Class
       
:param blah: Description of parameter blah
:type blah: bool

Note that the parameter has type int but Napoleon took the type from the attribute which has type bool.
A more practical reason this would happen is that the parameter has type Optional[dict] and this is stored into an attribute of type dict where if None is passed a default dictionary is created.

I added a new parameter is_attribute to _lookup_annotation and _consume_field(s) which defaults to False. If is_attribute is False and what is "class" then we look up the type from the constructor arguments instead. With these changes it produces the correct output:

:param blah: Description of parameter blah
:type blah: int

[jfbu]

merging into this the latest master update for CI checks