Linutronix / elbe

Embedded Linux Build Environment
https://elbe-rfs.org
GNU General Public License v3.0
170 stars 59 forks source link

tree-wide: use f-strings #351

Closed d4nuu8 closed 1 year ago

d4nuu8 commented 2 years ago

This series of patches replaces all "old-school" string formatting by f-strings.

This "fixes" all (668) pylint messages "consider-using-f-string / C0209".

See https://pylint.pycqa.org/en/latest/user_guide/messages/convention/consider-using-f-string.html

This improves pylint's rating from 8.73 to 9.28 out of 10.

d4nuu8 commented 2 years ago

@bgermann As requested on the mailing list.

I still need to check the problems you are having building a initvm with this applied.

d4nuu8 commented 2 years ago

Changes since last version:

diff.txt

@bgermann initvm creation and build are working now.

d4nuu8 commented 2 years ago

@bgermann May I ask for any feedback here?

bgermann commented 2 years ago

This will take time because I have to manually verify this manual approach.

bgermann commented 1 year ago

Integrated. Thanks for the contribution!