Skip to content

Conversation

@mukteshd
Copy link
Collaborator

@mukteshd mukteshd commented Oct 8, 2025

PR Details

Clickup Link - https://app.clickup.com/t/2323726/DIST-18283

Description

As part of updating rails app to use ruby 3.4.6, we needed to change ddrace gem with datadog since ddtrace was only compatible with ruby < 3.4

This change requires to replace dependency in datadog notifier gem from ddtrace to datadog

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation and I have updated the documentation accordingly.
  • I have added tests to cover my changes.

Copilot AI review requested due to automatic review settings October 8, 2025 08:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR replaces the ddtrace gem dependency with the datadog gem to support Ruby 3.4.6 compatibility, as ddtrace is not compatible with Ruby versions >= 3.4.

  • Updated gem dependency from ddtrace >= 1.13.0 to datadog >= 2.0.0
  • Updated gem summary to reflect the dependency change

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mukteshd mukteshd changed the title replace ddtrace with datadog for new ruby version replace ddtrace with datadog for new ruby version [v1.1.0] Oct 9, 2025
The gemspec was requiring datadog_notifier.rb which loads the datadog gem,
but this gem isn't available during bundle install phase. This caused
LoadError in CI when installing dependencies.

Solution: Only require datadog_notifier/version.rb in gemspec since it has
no external dependencies.
Copy link

@AjitKonde-Pattern AjitKonde-Pattern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants