FIX: coroutines can have a return statement

The value returned by the coroutine is carried on the `StopIteration` Exception
that is raised when exhausted.

Author: tacaswell

numpydoc/docscrape.py

@@ -394,12 +394,8 @@ def _parse(self):
         sections = list(self._read_sections())
         section_names = {section for section, content in sections}
 
-        has_returns = "Returns" in section_names
         has_yields = "Yields" in section_names
         # We could do more tests, but we are not. Arbitrarily.
-        if has_returns and has_yields:
-            msg = "Docstring contains both a Returns and Yields section."
-            raise ValueError(msg)
         if not has_yields and "Receives" in section_names:
             msg = "Docstring contains a Receives section but not Yields."
             raise ValueError(msg)

numpydoc/tests/test_docscrape.py

@@ -279,7 +279,9 @@ def test_returnyield():
     The number of bananas.
 
 """
-    assert_raises(ValueError, NumpyDocString, doc_text)
+    doc = NumpyDocString(doc_text)
+    assert len(doc['Returns']) == 1
+    assert len(doc['Yields']) == 2
 
 
 def test_section_twice():