MNT: Remove Python 2 leftovers#1116
Conversation
I actually prefer f-strings nowadays. Is it simple to change to that? |
I think so, using the See also the note in the linked docs which sounds good:
I'm going to do the f-strings in a separate commit so you can have a look. |
1b4dba3 to
7874ace
Compare
to improve readability and unify coding style (older code used %, newer contributions used format etc). Modifications were done using pyupgrade. Files in tinybuild\examples\future were deliberately left unchanged.
|
After rebasing I get two failing tests that pass locally so I assume it's not a real test failure. I can't "Rerun failed jobs" in https://dev.azure.com/sphinx-gallery/sphinx-gallery/_build/results?buildId=1246&view=results due to |
|
I usually |
|
Thanks @StefRe ! |
to improve readability and unify coding style (older code used
%, newer contributions useformatetc). Modifications were done using pyupgrade.Files in tinybuild\examples\future were deliberately left unchanged.
I know it's a lot of changed files but I think it greatly improves readability and the Python docs also recommend the use of
formatover%.