➜
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.
Created on 2021-10-16 18:42 by shivnaren, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Messages (4) | |||
|---|---|---|---|
| msg404108 - (view) | Author: Shivnaren Srinivasan (shivnaren) * | Date: 2021-10-16 18:42 | |
I use `argparse` a lot, and I think it's great—going through the source code, I see all string interpolation is with the old printf `%` formatting. AFAIK, f-strings are now the suggested method for interpolation (*after they were introduced, that is*). Do we want to refactor the '%' instances to f-strings? Happy to submit a PR if so. |
|||
| msg404113 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) *  |
Date: 2021-10-16 20:46 | |
Pure cosmetic changes are usually rejected. |
|||
| msg404114 - (view) | Author: Eric V. Smith (eric.smith) * |
Date: 2021-10-16 20:49 | |
As a rule, we don’t accept such wholesale changes. If there’s a performance problem, or a bug, then we’d look at individual cases. |
|||
| msg404116 - (view) | Author: Shivnaren Srinivasan (shivnaren) * | Date: 2021-10-16 21:33 | |
Understood--thank you for the responses. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:51 | admin | set | github: 89660 |
| 2021-10-16 21:33:33 | shivnaren | set | status: open -> closed resolution: rejected messages: + msg404116 stage: resolved |
| 2021-10-16 20:49:25 | eric.smith | set | nosy:
+ eric.smith messages: + msg404114 |
| 2021-10-16 20:46:11 | serhiy.storchaka | set | nosy:
+ serhiy.storchaka messages: + msg404113 |
| 2021-10-16 18:42:33 | shivnaren | create | |