Skip to content

Fix KDoc for RumMonitor#stopSession#2480

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/fix-kdoc-for-rummonitor-stopsession
Jan 9, 2025
Merged

Fix KDoc for RumMonitor#stopSession#2480
0xnm merged 1 commit into
developfrom
nogorodnikov/fix-kdoc-for-rummonitor-stopsession

Conversation

@0xnm

@0xnm 0xnm commented Jan 9, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Tiny change to use KDoc-specific syntax to properly reference existing code symbols.

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 January 9, 2025 12:28
@codecov-commenter

codecov-commenter commented Jan 9, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.86%. Comparing base (285ae49) to head (bd05ee7).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2480      +/-   ##
===========================================
+ Coverage    69.80%   69.86%   +0.05%     
===========================================
  Files          787      787              
  Lines        29469    29469              
  Branches      4920     4920              
===========================================
+ Hits         20570    20586      +16     
+ Misses        7528     7524       -4     
+ Partials      1371     1359      -12     
Files with missing lines Coverage Δ
.../main/kotlin/com/datadog/android/rum/RumMonitor.kt 28.57% <ø> (ø)

... and 35 files with indirect coverage changes

@0xnm
0xnm merged commit 7444761 into develop Jan 9, 2025
@0xnm
0xnm deleted the nogorodnikov/fix-kdoc-for-rummonitor-stopsession branch January 9, 2025 13:09
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