Message 285942 - Python tracker

Message285942

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.

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.

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