Skip to content

chore: split platform code per translation unit#255

Merged
dmehala merged 13 commits intomainfrom
dmehala/split-platform.cpp
Nov 19, 2025
Merged

chore: split platform code per translation unit#255
dmehala merged 13 commits intomainfrom
dmehala/split-platform.cpp

Conversation

@dmehala
Copy link
Collaborator

@dmehala dmehala commented Oct 29, 2025

Description

split platform code per translation unit for maintainability purposes.

@dmehala dmehala requested a review from a team as a code owner October 29, 2025 03:25
@dmehala dmehala requested review from dubloom and removed request for a team October 29, 2025 03:25
@pr-commenter
Copy link

pr-commenter bot commented Oct 29, 2025

Benchmarks

Benchmark execution time: 2025-11-19 13:20:43

Comparing candidate commit c9a8b2a in PR branch dmehala/split-platform.cpp with baseline commit 6dab714 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

@codecov-commenter
Copy link

codecov-commenter commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.79%. Comparing base (6dab714) to head (c9a8b2a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   87.80%   87.79%   -0.01%     
==========================================
  Files          84       84              
  Lines        5648     5646       -2     
==========================================
- Hits         4959     4957       -2     
  Misses        689      689              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@datadog-official

This comment has been minimized.

Base automatically changed from dmehala/v2-process-discovery to main November 11, 2025 20:14
@dmehala dmehala force-pushed the dmehala/split-platform.cpp branch from d27eb7e to 6553675 Compare November 19, 2025 13:09
@dmehala dmehala merged commit fad6060 into main Nov 19, 2025
25 checks passed
@dmehala dmehala deleted the dmehala/split-platform.cpp branch November 19, 2025 13:31
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