Enable ruff pyupgrade (UP) lint rules

[toroleapinc]

Summary

Enable a subset of safe pyupgrade (UP) ruff rules that are compatible with the Python 3.8 target version:

• UP011: Remove unnecessary parentheses from @lru_cache()
• UP015: Remove redundant open() mode arguments
• UP032: Replace .format() calls with f-strings
• UP034: Remove extraneous parentheses
• UP037: Remove unnecessary quoted annotations (in files with from __future__ import annotations)

All 42 violations were auto-fixed by ruff check --fix.

Not included (unsafe for py38 target): UP006, UP007, UP031, UP045 which require Python 3.9+/3.10+ syntax.

Part of #4227

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: lzchen / name: Leighton Chen (2a2d0f5)

[xrmx]

@toroleapinc thanks for the PR we can accept contributions only if you sign the CLA

[MikeGoldsmith]

Looks good to me 👍🏻

[tammy-baylis-swi]

Thanks @toroleapinc ! Please do these and push the fixes:

• tox -e precommit
• tox -e lint-opentelemetry-sdk

[lzchen]

@toroleapinc

Thanks for the contribution! Just curious, what was the thought-process behind choosing these specific ruff rules or is this arbitrary?

[toroleapinc]

@lzchen These weren't arbitrary — I picked the rules that had the most existing violations in the codebase to get the biggest cleanup in one pass:

• UP032 (f-string): most of the diff, converting .format() calls to f-strings for readability
• UP037 (quoted-annotation): removing unnecessary string quotes around forward-ref annotations that are no longer needed
• UP011 (lru-cache-without-parameters): minor cleanup, @lru_cache() → @lru_cache
• UP015 (redundant-open-modes): dropping explicit "r" mode in open() calls
• UP034 (extraneous-parentheses): removing redundant parens

I deliberately left out the more disruptive UP rules (like UP007 for X | Y union syntax) since those would require bumping the minimum Python version. Happy to adjust the scope if you'd prefer a different subset.

[toroleapinc]

@tammy-baylis-swi Ran both — tox -e precommit caught some whitespace and formatting issues, pushed the fixes. tox -e lint-opentelemetry-sdk passes (10/10 rating) but has a pre-existing E1101 on measurement_consumer.py:67 that's unrelated to this PR.

[lzchen]

@toroleapinc

Happy to adjust the scope if you'd prefer a different subset.

No need, the separation makes sense. As long as we have a documented set of supported/supported SGTM.

[xrmx]

@toroleapinc In another PR we bumped the baseline to 3.9 so feel free to open a new PR with 3.9+ safe rules if you want