Skip to content

RUM-10672: Stop telemetry for compose checkbox and radiobutton#2775

Merged
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-10672/reduce-compose-telemetry
Jul 3, 2025
Merged

RUM-10672: Stop telemetry for compose checkbox and radiobutton#2775
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-10672/reduce-compose-telemetry

Conversation

@jonathanmos

Copy link
Copy Markdown
Member

What does this PR do?

The reflection by which we get attributes for radiobuttons and checkboxes in Compose is fragile and generates a lot of telemetry. Until we can find an alternative method to get the attributes, this PR removes the telemetry for these components.

Motivation

Volume of telemetry generated.

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@codecov-commenter

codecov-commenter commented Jul 3, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.83%. Comparing base (2e51912) to head (92dd20a).
⚠️ Report is 1930 commits behind head on develop.

Files with missing lines Patch % Lines
...y/compose/internal/reflection/ComposeReflection.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2775      +/-   ##
===========================================
- Coverage    70.03%   69.83%   -0.20%     
===========================================
  Files          824      824              
  Lines        30877    30877              
  Branches      5196     5196              
===========================================
- Hits         21623    21562      -61     
- Misses        7818     7858      +40     
- Partials      1436     1457      +21     
Files with missing lines Coverage Δ
...y/compose/internal/reflection/ComposeReflection.kt 0.00% <0.00%> (ø)

... and 29 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jonathanmos
jonathanmos marked this pull request as ready for review July 3, 2025 11:10
@jonathanmos
jonathanmos requested review from a team as code owners July 3, 2025 11:10
@jonathanmos
jonathanmos merged commit 6ce396b into develop Jul 3, 2025
26 checks passed
@jonathanmos
jonathanmos deleted the jmoskovich/rum-10672/reduce-compose-telemetry branch July 3, 2025 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants