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

Fix execution spec with cucumber on ruby 3.4 #4128

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Conversation

p-datadog
Copy link
Contributor

@p-datadog p-datadog commented Nov 18, 2024

What does this PR do?
Cucumber is broken on ruby 3.4 currently, requires the fix in cucumber/cucumber-ruby#1757 (not released yet).

Use their master from git on ruby 3.4 and unpend the test.

Motivation:
Investigating intermittent failure in that test on the other Ruby versions.

Change log entry
None

Additional Notes:

How to test the change?

Existing test is unpended.

@p-datadog p-datadog requested a review from a team as a code owner November 18, 2024 20:16
@github-actions github-actions bot added the dev/testing Involves testing processes (e.g. RSpec) label Nov 18, 2024
Cucumber is broken on ruby 3.4 currently, requires the fix in
cucumber/cucumber-ruby#1757
(not released yet).

Use their master from git on ruby 3.4 and unpend the test.
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.79%. Comparing base (672f309) to head (5e06e38).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4128      +/-   ##
==========================================
+ Coverage   97.77%   97.79%   +0.01%     
==========================================
  Files        1350     1350              
  Lines       81278    81276       -2     
  Branches     4108     4107       -1     
==========================================
+ Hits        79471    79481      +10     
+ Misses       1807     1795      -12     

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


🚨 Try these New Features:

Copy link
Contributor

@sarahchen6 sarahchen6 left a comment

Choose a reason for hiding this comment

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

👍👍

@pr-commenter
Copy link

pr-commenter bot commented Nov 18, 2024

Benchmarks

Benchmark execution time: 2024-11-18 21:03:50

Comparing candidate commit 5e06e38 in PR branch cucumber-ruby-3.4 with baseline commit 672f309 in branch master.

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

@p-datadog p-datadog merged commit 9b97769 into master Nov 18, 2024
283 checks passed
@p-datadog p-datadog deleted the cucumber-ruby-3.4 branch November 18, 2024 21:25
@github-actions github-actions bot added this to the 2.8.0 milestone Nov 18, 2024
@p-datadog p-datadog added the dev/internal Other internal work that does not need to be included in the changelog label Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/internal Other internal work that does not need to be included in the changelog dev/testing Involves testing processes (e.g. RSpec)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants