On Sun, May 15, 2022 at 02:18:53PM -0700, Ian Rogers wrote:
[...]
This looks good to me and will run on python 2. The code would be more idiomatic in python3 using f-strings. I'd rather the code was idiomatic from the beginning, but others may disagree and prefer python 2 compatibility (python 2 is now end of life). f-strings are python 3.6+ and so have been supported for 5 years.
Using f-string is the right thing for me, will update.
Thanks for reviewing and suggestion!
Leo