Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUM-4358: Log warning about tag modification only once #2017

Conversation

0xnm
Copy link
Contributor

@0xnm 0xnm commented May 2, 2024

What does this PR do?

This PR addresses issue #1018: with SDK v2 functionality we can now log a particular message only once. This is useful to avoid logcat spamming.

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)

@0xnm 0xnm requested review from a team as code owners May 2, 2024 07:40
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.60%. Comparing base (2adb85c) to head (be760da).

Additional details and impacted files
@@                Coverage Diff                @@
##           release/2.9.0    #2017      +/-   ##
=================================================
- Coverage          83.92%   83.60%   -0.32%     
=================================================
  Files                488      488              
  Lines              17781    17754      -27     
  Branches            2667     2665       -2     
=================================================
- Hits               14922    14842      -80     
- Misses              2154     2190      +36     
- Partials             705      722      +17     
Files Coverage Δ
...android/core/constraints/DatadogDataConstraints.kt 91.84% <100.00%> (+0.08%) ⬆️

... and 24 files with indirect coverage changes

@0xnm 0xnm merged commit c0eb3d8 into release/2.9.0 May 2, 2024
21 checks passed
@0xnm 0xnm deleted the nogorodnikov/rum-4358/log-message-about-tag-modification-only-once branch May 2, 2024 11:36
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.

None yet

4 participants