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. |
|
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 |
|