Add 'of' to GoogleDocstring multiple type (#2884)

A docstring of the following form should pass (see https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_numpy.html, search for ':obj:list of :obj:str')

def my_func(self):
    """This is a docstring.

    Returns
    -------
    :obj:`list` of :obj:`str`
        List of strings
    """
    return ["hi", "bye"] #@

CONTRIBUTORS.txt

@@ -294,3 +294,6 @@ contributors:
 * Taewon Kim : contributor
 
 * Daniil Kharkov: contributor
+
+* Zeb Nicholls: contributor
+    - Made W9011 compatible with 'of' syntax in return types

ChangeLog

@@ -7,6 +7,8 @@ What's New in Pylint 2.4.0?
 
 Release date: TBA
 
+* Allow of in `GoogleDocstring.re_multiple_type`
+
 * Added `subprocess-run-check` to handle subrocess.run without explicitly set `check` keyword.
 
   Close #2848

doc/whatsnew/2.4.rst

@@ -91,3 +91,17 @@ The following imports do not trigger an ``ungrouped-imports`` anymore ::
     import zipfile
     from unittest import TestCase
     from unittest.mock import MagicMock
+    
+* The checker for missing return documentation is now more flexible.
+
+The following does not trigger a ``missing-return-doc`` anymore ::
+
+    def my_func(self):
+        """This is a docstring.
+
+        Returns
+        -------
+        :obj:`list` of :obj:`str`
+            List of strings
+        """
+        return ["hi", "bye"] #@

pylint/extensions/_check_docs_utils.py

@@ -443,7 +443,7 @@ class GoogleDocstring(Docstring):
 
     re_multiple_type = r"""
         (?:{container_type}|{type}|{xref})
-        (?:\s+or\s+(?:{container_type}|{type}|{xref}))*
+        (?:\s+(?:of|or)\s+(?:{container_type}|{type}|{xref}))*
     """.format(
         type=re_type, xref=re_xref, container_type=re_container_type
     )

pylint/test/extensions/test_check_return_docs.py

@@ -250,6 +250,21 @@ def my_func(self):
         with self.assertNoMessages():
             self.checker.visit_return(return_node)
 
+    def test_find_numpy_returns_with_of(self):
+        return_node = astroid.extract_node('''
+        def my_func(self):
+            """This is a docstring.
+
+            Returns
+            -------
+            :obj:`list` of :obj:`str`
+                List of strings
+            """
+            return ["hi", "bye"] #@
+        ''')
+        with self.assertNoMessages():
+            self.checker.visit_return(return_node)
+
     def test_ignores_sphinx_return_none(self):
         return_node = astroid.extract_node('''
         def my_func(self, doc_type):