classification
Title: fix OrderedDict.__init__ docstring to reflect PEP 468
Type: Stage:
Components: Library (Lib) Versions: Python 3.7, Python 3.6
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: rhettinger Nosy List: brett.cannon, eric.snow, jonathaneunice, rhettinger, serhiy.storchaka
Priority: normal Keywords:

Created on 2017-06-14 04:46 by jonathaneunice, last changed 2017-06-15 00:38 by jonathaneunice.

Pull Requests
URL Status Linked Edit
PR 2179 open jonathaneunice, 2017-06-14 05:01
Messages (4)
msg295982 - (view) Author: Jonathan Eunice (jonathaneunice) * Date: 2017-06-14 04:46
In Python 3.6 and current repo (pre-3.7), the docstring for collections.OrderedDict.__init__() stats that keyword argument order is arbitrary. That is untrue if I properly understand PEP 468, which as of Python 3.6 makes the order of kwargs stable / identical to the order seen in the source code.

Associated PR imminent.
msg295987 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-06-14 05:26
Currently there are two implementations of OrderedDict, and Python implementation is not used. Is C implementation need an update? What about the module documentation, is it need an update?

I don't think that we should include such details about past Python versions in docstrings. I can't remember precedences.
msg296039 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2017-06-14 21:00
Past Python details should not be included in docstrings; it should simply represent the current code.
msg296053 - (view) Author: Jonathan Eunice (jonathaneunice) * Date: 2017-06-15 00:38
PR updated to incorporate Serhiy Storchaka and Brett Cannon feedback.
History
Date User Action Args
2017-06-15 00:38:48jonathaneunicesetmessages: + msg296053
2017-06-14 21:00:30brett.cannonsetnosy: + eric.snow, brett.cannon
messages: + msg296039
2017-06-14 05:26:18serhiy.storchakasetmessages: + msg295987
2017-06-14 05:15:24rhettingersetnosy: + serhiy.storchaka
2017-06-14 05:15:13rhettingersetassignee: rhettinger

nosy: + rhettinger
versions: + Python 3.6
2017-06-14 05:01:15jonathaneunicesetpull_requests: + pull_request2232
2017-06-14 04:46:11jonathaneunicecreate