TkinterWeb icon indicating copy to clipboard operation
TkinterWeb copied to clipboard

Some tidyups while using the project

Open marksmayo opened this issue 2 years ago • 2 comments

Updated deprecated call Updated format strings to f-strings Tidied spaces and reordered imports.

marksmayo avatar Sep 12 '23 04:09 marksmayo

Hey! Thanks for the pull request. I've been hesitant to use f-strings in an effort to maintain Python 2 support but seeing as Python 2 has been deprecated for two years now it might be time to move on. I'll think about this?

Andereoo avatar Sep 13 '23 13:09 Andereoo

Yeah I saw afterwards you support 2.6, and wondered, but I also would (personally) be surprised at new projects using it, and existing users on 2.6 are likely used to having to stick to old releases of libraries...maybe?

Anyway, all good, I was playing with it and I like to just do my little bit for a project when learning about it :)

marksmayo avatar Sep 13 '23 22:09 marksmayo

@marksmayo I finally merged your changes. At this point no one should still be using Python 2 and still expecting packages to still support it. Thanks again for the tidyups!

Andereoo avatar Jul 24 '24 00:07 Andereoo