Skip to content

[PROF-15559] Update CI now that we no longer use MJIT headers - #32

Merged
ivoanjo merged 1 commit into
masterfrom
ivoanjo/prof-15559-fix-outdated-ci-comments
Aug 18, 2026
Merged

[PROF-15559] Update CI now that we no longer use MJIT headers#32
ivoanjo merged 1 commit into
masterfrom
ivoanjo/prof-15559-fix-outdated-ci-comments

Conversation

@ivoanjo

@ivoanjo ivoanjo commented Aug 18, 2026

Copy link
Copy Markdown
Member

What does this PR do?

This PR is a follow-up to
#29 and updates our CI to test all Ruby versions from 2.5 to 4.0, and assume that all use the headers shipped in this gem.

Motivation:

Make our CI reflect the change.

Additional Notes:

This PR will be red and can only be merged after DataDog/dd-trace-rb#6135 lands.

How to test the change?

CI will be green ;)

**What does this PR do?**

This PR is a follow-up to
#29
and updates our CI to test all Ruby versions from 2.5 to 4.0, and
assume that all use the headers shipped in this gem.

**Motivation:**

Make our CI reflect the change.

**Additional Notes:**

This PR will be red and can only be merged after
DataDog/dd-trace-rb#6135 lands.

**How to test the change?**

CI will be green ;)
@ivoanjo
ivoanjo requested a review from eregon August 18, 2026 14:02
@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Aug 18, 2026

Copy link
Copy Markdown

Pipelines

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 7816aba | Docs | View more details | Give us feedback!

@ivoanjo

ivoanjo commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

DataDog/dd-trace-rb#6135 has landed, poking CI to retry...

@ivoanjo
ivoanjo enabled auto-merge August 18, 2026 16:09
@ivoanjo
ivoanjo merged commit 1416276 into master Aug 18, 2026
81 of 101 checks passed
@ivoanjo
ivoanjo deleted the ivoanjo/prof-15559-fix-outdated-ci-comments branch August 18, 2026 16:11
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.

2 participants