The committers listed above are authorized under a signed CLA.

• ✅ login: DominicOram / name: Dominic Oram (69bc2c4, bc36e5b, 9db7882, ad00d07)
• ✅ login: emdneto / name: Emídio Neto (5509a87)
• ✅ login: xrmx / name: Riccardo Magliocchetti (c18e48d, d7aa808, 338ee42, e966d7d, b1f2c40)

If you add from __future__ import annotations you can avoid importing Optional and use | None syntax, while you are at it you may also drop the usage of Dict and List and use dict and list instead.

My reasoning was that the Optional style is what's currently used in most places and I wanted to keep consistency but happy to change to this.

If you add from __future__ import annotations you can avoid importing Optional and use | None syntax, while you are at it you may also drop the usage of Dict and List and use dict and list instead.

Updated the places where I have changed something and a few places in the same area to keep some consistency. I would be happy to do more but feels like a new PR as it would be a big change and I would rather get my original issue fixed sooner

I've filed this open-telemetry/opentelemetry-python-contrib#3320 because this PR failed at least three times that test. Since this continues to fail maybe it's not a flaky test and we are introducing a regression somewhere here instead.