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

Replace 'debase' with 'datadog'-ruby_core_source #4014

Merged
merged 5 commits into from
Oct 22, 2024

Conversation

sarahchen6
Copy link
Contributor

@sarahchen6 sarahchen6 commented Oct 18, 2024

Change log entry
Replace the debase-ruby_core_source gem with the datadog-ruby_core_source.

What does this PR do?
This PR replaces the debase-ruby_core_source gem with the datadog-ruby_core_source gem.

Motivation:
The goal of this PR is to update the library such that it now uses the datadog-ruby_core_source Ruby gem.

Additional Notes:
This PR is a step toward the overall project goal of creating a new datadog-ruby_core_source gem. See here for more info on the project.

How to test the change?
TBD

Unsure? Have a question? Request a review!

@sarahchen6 sarahchen6 force-pushed the sarahchen6/update-ruby-core-soure-gem branch 3 times, most recently from 985ce62 to 8da89c9 Compare October 18, 2024 20:48
@sarahchen6 sarahchen6 changed the title WIP Replace 'debase' with 'datadog'-ruby_core_source Oct 18, 2024
datadog.gemspec Outdated Show resolved Hide resolved
@sarahchen6 sarahchen6 closed this Oct 21, 2024
@sarahchen6 sarahchen6 deleted the sarahchen6/update-ruby-core-soure-gem branch October 21, 2024 20:54
@sarahchen6 sarahchen6 restored the sarahchen6/update-ruby-core-soure-gem branch October 21, 2024 20:55
@sarahchen6 sarahchen6 reopened this Oct 21, 2024
@pr-commenter
Copy link

pr-commenter bot commented Oct 21, 2024

Benchmarks

Benchmark execution time: 2024-10-22 14:00:55

Comparing candidate commit 9743f03 in PR branch sarahchen6/update-ruby-core-soure-gem with baseline commit df97f48 in branch master.

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

@sarahchen6 sarahchen6 force-pushed the sarahchen6/update-ruby-core-soure-gem branch from fc7dd3c to 379eb47 Compare October 21, 2024 21:36
@DataDog DataDog deleted a comment from codecov-commenter Oct 21, 2024
@codecov-commenter
Copy link

codecov-commenter commented Oct 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.86%. Comparing base (df97f48) to head (9743f03).
Report is 8 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4014   +/-   ##
=======================================
  Coverage   97.86%   97.86%           
=======================================
  Files        1319     1319           
  Lines       79141    79141           
  Branches     3927     3927           
=======================================
  Hits        77449    77449           
  Misses       1692     1692           

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

@sarahchen6 sarahchen6 marked this pull request as ready for review October 22, 2024 00:24
@sarahchen6 sarahchen6 requested review from a team as code owners October 22, 2024 00:24
Copy link
Contributor

@delner delner left a comment

Choose a reason for hiding this comment

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

Code changes look good 👍

Copy link
Member

@ivoanjo ivoanjo left a comment

Choose a reason for hiding this comment

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

👍 This looks great, thank you!

Copy link
Member

@Strech Strech left a comment

Choose a reason for hiding this comment

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

👏🏼 amazing job!

@TonyCTHsu TonyCTHsu enabled auto-merge (rebase) October 22, 2024 11:29
@TonyCTHsu TonyCTHsu force-pushed the sarahchen6/update-ruby-core-soure-gem branch from 80d83be to ccc12f4 Compare October 22, 2024 13:14
@TonyCTHsu TonyCTHsu disabled auto-merge October 22, 2024 13:30
@TonyCTHsu TonyCTHsu merged commit e20ee5b into master Oct 22, 2024
275 of 276 checks passed
@TonyCTHsu TonyCTHsu deleted the sarahchen6/update-ruby-core-soure-gem branch October 22, 2024 14:01
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.

6 participants