Skip to content

Fix proguard rules for Method Called Metrics#2691

Merged
ambushwork merged 1 commit into
developfrom
yl/fix-metric-proguard
Jun 3, 2025
Merged

Fix proguard rules for Method Called Metrics#2691
ambushwork merged 1 commit into
developfrom
yl/fix-metric-proguard

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

We have "Method Called" Metrics in both RUM DatadogRumMonitor class and Session Replay TreeViewTraversal class,
their class name will be assigned as the attribute of "caller_class". However they can be obfuscated in release build, so we need to put them in the "consumer-rules.pro" file to keep the class name.

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 Jun 2, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.96%. Comparing base (7f6ef9e) to head (e0f0bcf).
⚠️ Report is 2054 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2691      +/-   ##
===========================================
+ Coverage    69.89%   69.96%   +0.06%     
===========================================
  Files          822      822              
  Lines        30692    30692              
  Branches      5169     5169              
===========================================
+ Hits         21452    21471      +19     
+ Misses        7793     7769      -24     
- Partials      1447     1452       +5     

see 30 files with indirect coverage changes

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

@ambushwork
ambushwork marked this pull request as ready for review June 2, 2025 09:37
@ambushwork
ambushwork requested review from a team as code owners June 2, 2025 09:37
@ambushwork
ambushwork merged commit 3f5f691 into develop Jun 3, 2025
@ambushwork
ambushwork deleted the yl/fix-metric-proguard branch June 3, 2025 06:52
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.

3 participants