python-novice-inflammation icon indicating copy to clipboard operation
python-novice-inflammation copied to clipboard

fix: use normal print instead of percent formatting

Open henryiii opened this issue 2 years ago • 1 comments

This was using the old percent formatting instead of the more modern .format (Python 2.6+) or f-strings (Python 3.6+). What's more, string formatting is not introduced or discussed anywhere else in the lesson, and it is not elaborated here. I think the simplest thing for this is to simply not introduce it here, it's not needed, and multi-argument print calls have been introduced.

henryiii avatar Jun 10 '22 14:06 henryiii

Hi, @henryiii. The change you propose here makes sense to me and IMHO can be merged as is. Unfortunately, I no longer have write access to this repository and I don't know who or when can look at / merge this PR.

CC @fmichonneau

maxim-belkin avatar Jun 10 '22 19:06 maxim-belkin

I too agree with you. Merging this PR @henryiii @maxim-belkin Thanks for bringing this up and contributing towards the same! :)

ineelhere avatar Sep 30 '22 12:09 ineelhere