Skip to content

RUM-8349: Add support for detekt checks for session replay compose module#2507

Merged
ambushwork merged 1 commit into
developfrom
yl/fix-detekt-modifier
Feb 19, 2025
Merged

RUM-8349: Add support for detekt checks for session replay compose module#2507
ambushwork merged 1 commit into
developfrom
yl/fix-detekt-modifier

Conversation

@ambushwork

@ambushwork ambushwork commented Feb 18, 2025

Copy link
Copy Markdown
Member

What does this PR do?

  • Add detekt for dd-sdk-android-session-replay-compose in gitlab-ci.yml
  • Fix detekt_custom.yml for this module

Motivation

RUM-8349

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)

@ambushwork
ambushwork force-pushed the yl/fix-detekt-modifier branch 2 times, most recently from a25b286 to c8c02ca Compare February 18, 2025 12:51
@ambushwork
ambushwork force-pushed the yl/fix-detekt-modifier branch from c8c02ca to 29548cc Compare February 18, 2025 13:48
@ambushwork ambushwork changed the title RUM-8349: Add support for detekt checks for session replay compose mo… RUM-8349: Add support for detekt checks for session replay compose module Feb 18, 2025
@codecov-commenter

codecov-commenter commented Feb 18, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.98%. Comparing base (85637ab) to head (29548cc).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2507      +/-   ##
===========================================
- Coverage    69.99%   69.98%   -0.01%     
===========================================
  Files          794      794              
  Lines        29875    29875              
  Branches      4991     4991              
===========================================
- Hits         20911    20907       -4     
+ Misses        7584     7582       -2     
- Partials      1380     1386       +6     

see 33 files with indirect coverage changes

@ambushwork
ambushwork marked this pull request as ready for review February 18, 2025 14:23
@ambushwork
ambushwork requested review from a team as code owners February 18, 2025 14:23
@satween
satween self-requested a review February 19, 2025 14:48
@ambushwork
ambushwork merged commit fe2a4ca into develop Feb 19, 2025
@ambushwork
ambushwork deleted the yl/fix-detekt-modifier branch February 19, 2025 15:05
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