Message 285942 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	methane
Recipients	Arfrever, Julien.Palard, Ramchandra Apte, berker.peksag, brian.curtin, eric.araujo, eric.snow, ezio.melotti, gregory.p.smith, gruszczy, martin.panter, meatballhat, methane, nchammas, rhettinger, serhiy.storchaka
Date	2017-01-21.07:13:37
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1484982817.97.0.0811145300936.issue8706@psf.upfronthosting.co.za>
In-reply-to

Content
TL;DR Changing positional argument name doesn't break backward compatibility. After start accepting keyword argument, it's name is part of API and should be stable. For example, document says `str.startswith(prefix[, start[, end]])`. But this patch seems using `sub` instead of `prefix`. https://docs.python.org/3.5/library/stdtypes.html#str.startswith Keyword name should be chosen carefully, like choosing method name. So I'm -1 to adding keyword argument support to many method in one issue.

TL;DR

Changing positional argument name doesn't break backward compatibility.
After start accepting keyword argument, it's name is part of API and should be stable.

For example, document says `str.startswith(prefix[, start[, end]])`.
But this patch seems using `sub` instead of `prefix`.
https://docs.python.org/3.5/library/stdtypes.html#str.startswith

Keyword name should be chosen carefully, like choosing method name.
So I'm -1 to adding keyword argument support to many method in one issue.

History
Date	User	Action	Args
2017-01-21 07:13:38	methane	set	recipients: + methane, rhettinger, gregory.p.smith, ezio.melotti, eric.araujo, Arfrever, brian.curtin, gruszczy, meatballhat, eric.snow, Ramchandra Apte, berker.peksag, martin.panter, serhiy.storchaka, Julien.Palard, nchammas
2017-01-21 07:13:37	methane	set	messageid: <1484982817.97.0.0811145300936.issue8706@psf.upfronthosting.co.za>
2017-01-21 07:13:37	methane	link	issue8706 messages
2017-01-21 07:13:37	methane	create