Skip to content

chore(ci): build with sanitizers #252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore(ci): build with sanitizers #252

wants to merge 2 commits into from

Conversation

@dmehala
Copy link
Collaborator

@dmehala dmehala commented Oct 22, 2025

Description

Now, the CI builds and tests the library with sanitizers enabled.

@dmehala dmehala requested a review from a team as a code owner October 22, 2025 11:19
@dmehala dmehala requested review from dubloom and zacharycmontoya and removed request for a team October 22, 2025 11:19
@pr-commenter
Copy link

pr-commenter bot commented Oct 22, 2025 •
edited
Loading

Benchmarks

Benchmark execution time: 2025-10-22 18:16:00

Comparing candidate commit 27f12e1 in PR branch dmehala/enable-sanitizers with baseline commit 0c13dba in branch main.

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

zacharycmontoya
zacharycmontoya reviewed Oct 22, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zacharycmontoya zacharycmontoya zacharycmontoya approved these changes

@dubloom dubloom Awaiting requested review from dubloom dubloom is a code owner automatically assigned from DataDog/dd-trace-cpp

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dmehala @zacharycmontoya