Skip to content

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

Merged
0xnm merged 1 commit into
release/2.9.0from
nogorodnikov/rum-4358/log-message-about-tag-modification-only-once
May 2, 2024
Merged

RUM-4358: Log warning about tag modification only once#2017
0xnm merged 1 commit into
release/2.9.0from
nogorodnikov/rum-4358/log-message-about-tag-modification-only-once

Conversation

@0xnm

@0xnm 0xnm commented May 2, 2024

Copy link
Copy Markdown
Member

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

codecov-commenter commented May 2, 2024

Copy link
Copy Markdown

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
@0xnm
0xnm deleted the nogorodnikov/rum-4358/log-message-about-tag-modification-only-once branch May 2, 2024 11:36
@xgouchet xgouchet added this to the 2.9.X milestone Jul 31, 2024
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