Skip to content
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.

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

(Draft) [NO-TICKET] Update crashtracker with libdatadog 15 breaking changes #4255

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ivoanjo
Copy link
Member

@ivoanjo ivoanjo commented Jan 3, 2025

What does this PR do?

This PR updates the crasktracker C code to build with the latest libdatadog changes (in main) that will become part of libdatadog 15.

Motivation:

I'm working on a libdatadog branch and had to do this to unblock my work, so I decided I'll create a PR with it so nobody needs to repeat this work.

Change log entry

No

Additional Notes:

I'm opening this PR as draft as we shouldn't merge this until libdatadog 15 is out.

How to test the change?

Existing test coverage is enough to validate this.

**What does this PR do?**

This PR updates the crasktracker C code to build with the latest
libdatadog changes (in main) that will become part of libdatadog 15.

**Motivation:**

I'm working on a libdatadog branch and had to do this to unblock
my work, so I decided I'll create a PR with it so nobody needs to
repeat this work.

**Additional Notes:**

I'm opening this PR as draft as we shouldn't merge this until
libdatadog 15 is out.

**How to test the change?**

Existing test coverage is enough to validate this.
@ivoanjo ivoanjo added core Involves Datadog core libraries dev/internal Other internal work that does not need to be included in the changelog labels Jan 3, 2025
@datadog-datadog-prod-us1
Copy link
Contributor

datadog-datadog-prod-us1 bot commented Jan 3, 2025

Datadog Report

Branch report: ivoanjo/libdatadog-15-crashtracker
Commit report: 1ec8a96
Test service: dd-trace-rb

✅ 0 Failed, 21336 Passed, 1940 Skipped, 5m 31.41s Total Time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Involves Datadog core libraries dev/internal Other internal work that does not need to be included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant