Fix: Move tags from transaction.contexts.trace.tags to transaction.tags

[maciejwalkowiak]

📜 Description

Move tags from transaction.contexts.trace.tags to transaction.tags.

💡 Motivation and Context

Fixes #1693

The implementation could be cleaner - like introducing a dedicated TraceContext object, but this would mean we introduce a breaking change - something to keep in mind for 6.0.

💚 How did you test it?

Unit tests.

📝 Checklist

• I reviewed the submitted code
• I added tests to verify the changes
• I updated the docs if needed
• No breaking changes

[codecov-commenter]

Codecov Report

Merging #1700 (de6be68) into main (717ccad) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #1700      +/-   ##
============================================
+ Coverage     74.80%   74.90%   +0.10%     
- Complexity     2077     2081       +4     
============================================
  Files           210      210              
  Lines          7430     7445      +15     
  Branches        782      783       +1     
============================================
+ Hits           5558     5577      +19     
+ Misses         1491     1487       -4     
  Partials        381      381              

Impacted Files	Coverage Δ
sentry/src/main/java/io/sentry/SpanContext.java	93.75% <100.00%> (+0.56%)	⬆️
...ain/java/io/sentry/protocol/SentryTransaction.java	92.50% <100.00%> (+2.84%)	⬆️
sentry/src/main/java/io/sentry/SentryTracer.java	89.20% <0.00%> (+1.43%)	⬆️
sentry/src/main/java/io/sentry/Span.java	95.08% <0.00%> (+3.27%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 717ccad...de6be68. Read the comment docs.

[marandaneto]

left a comment but rather than that LGTM, thanks @maciejwalkowiak