gh-132390: Apply Ruff linting to Tools/build

[picnixz]

• Issue: Lint Tools/build using existing ruff configuration #132390

[picnixz]

@AA-Turner @hugovk Does it look acceptable or would you prefer to just exclude the files instead of ruffing them?

[picnixz]

I'll actually hold the merge a bit more so that code owners can also give their opinion.

[AlexWaygood]

Looks like a nice cleanup to me, but I'm not the person maintaining most of these scripts :-)

make sure to get signoff from the relevant maintainers before landing this. Not everybody on the team is necessarily on board with stricter linting in CI!

[picnixz]

make sure to get signoff from the relevant maintainers before landing this. Not everybody on the team is necessarily on board with stricter linting in CI!

Yes, that's why I eventually decided to hold off the merge!

[picnixz]

Ok so here is how I decided to make the change:

• use f-strings instead of .format(); both have the same readability IMO.
• do not use f-strings for %s when it's less readable. And if there are enough occurrences of %-usage, just ignore it for the file (namely generate_token.py)
• if %-style and f-strings are roughly the same readability and the file already uses f-strings elsewhere, make everything f-strings (smelly.py)

[sethmlarson]

LGTM for generate_sbom.py.

[encukou]

Here, I did want to point out that it's a single string and not a forgotten comma.
Could you point me to the reasoning behind this change?