Skip to content

Skip profiling schema files in session replay module#3119

Merged
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/skip-profiling-schemas-in-session-replay
Jan 8, 2026
Merged

Skip profiling schema files in session replay module#3119
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/skip-profiling-schemas-in-session-replay

Conversation

@aleksandr-gringauz

@aleksandr-gringauz aleksandr-gringauz commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Addressing this issue. We don't need schema files related to profiling in the session replay module.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

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)

@aleksandr-gringauz
aleksandr-gringauz force-pushed the aleksandr-gringauz/skip-profiling-schemas-in-session-replay branch from 0e235bb to 3864045 Compare January 8, 2026 13:37
@aleksandr-gringauz
aleksandr-gringauz marked this pull request as ready for review January 8, 2026 13:38
@aleksandr-gringauz
aleksandr-gringauz requested a review from a team as a code owner January 8, 2026 13:38
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Jan 8, 2026

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 65.79% (-0.07%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 3864045 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.76%. Comparing base (6845b3b) to head (3864045).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3119      +/-   ##
===========================================
- Coverage    70.78%   70.76%   -0.02%     
===========================================
  Files          893      893              
  Lines        33000    33000              
  Branches      5549     5549              
===========================================
- Hits         23357    23351       -6     
+ Misses        8096     8092       -4     
- Partials      1547     1557      +10     

see 36 files with indirect coverage changes

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

@aleksandr-gringauz

Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jan 8, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-01-08 15:18:57 UTC ℹ️ Start processing command /merge


2026-01-08 15:19:11 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-01-08 15:56:36 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in develop is approximately 1h (p90).


2026-01-08 16:02:28 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@aleksandr-gringauz
aleksandr-gringauz merged commit f1ab09f into develop Jan 8, 2026
27 checks passed
@aleksandr-gringauz
aleksandr-gringauz deleted the aleksandr-gringauz/skip-profiling-schemas-in-session-replay branch January 8, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants