[issue45733] importlib.abc.Traversable.name does not match

2021-11-05 Thread John-Mark Gurney
New submission from John-Mark Gurney : The documentation for Traversable.name says it is a method, not a property: https://docs.python.org/3/library/importlib.html#importlib.abc.Traversable.name The issue is that with Python 3.9.7 (default, Nov 1 2021, 11:26:33), using a standard posix

[issue42414] unable to document fields of dataclass

2021-07-20 Thread John-Mark Gurney
John-Mark Gurney added the comment: So, just looked at the patch, but it's missing the documentation part of it. Also, yes, you can add the doc as another line, but now that's two lines (yes, you can add semicolons to make it one line, but that might surprise some people). I

[issue42414] unable to document fields of dataclass

2021-07-20 Thread John-Mark Gurney
John-Mark Gurney added the comment: Though this suggestion does work, I am not a fan of this solution. The issue is that it separates the doc from the definition. This works well if you have only a field fields in the class, But if you get 10-20+ fields, it moves away the docs and makes it

[issue42414] unable to document fields of dataclass

2020-11-20 Thread John-Mark Gurney
John-Mark Gurney added the comment: As I said, I expect it to work similar to how property works: ``` >>> class Foo: ... def getx(self): ... return 5 ... x = property(getx, doc='document the x property') ... >>> help(Foo) Help on class Foo in module __main__

[issue42414] unable to document fields of dataclass

2020-11-19 Thread John-Mark Gurney
New submission from John-Mark Gurney : per: https://bugs.python.org/issue38401 There is not a way to document fields of a dataclass. I propose that instead of making a language change, that an additional parameter to the field be added in similar vein to property. This currently works

[issue39386] getting invalid data from async iterator

2020-01-18 Thread John-Mark Gurney
New submission from John-Mark Gurney : If I create a coro from an async iterator, then wait_for it w/ a timeout, but shielded, so it won't get canceled, and then await upon it, it returns invalid data. See the attached test case. The reason I do the following is to make sure that an

[issue1777134] minidom pretty xml output improvement

2008-02-12 Thread John-Mark Gurney
John-Mark Gurney added the comment: I think this is a good patch. It gives more useful pretty XML output. I would suggest that possibly this routine be moved to xml.dom or xml.dom.utils instead of being part of minidom since it should not be minidom specific. There is one bug in the patch in