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

adding minitest #6

Merged
merged 1 commit into from
Aug 18, 2023
Merged

adding minitest #6

merged 1 commit into from
Aug 18, 2023

Conversation

anmarchenko
Copy link
Member

@anmarchenko anmarchenko commented Aug 17, 2023

This PR is the next step to bring datadog-ci on par with ddtrace:

  • adding minitest integration (originally authored by @bravehager)
  • generating appraisal gemfiles for minitest-5
  • adding minitests specs to Rakefile
  • making sure that original implementation also instruments Minitest::Spec out of the box
  • adding docs on minitest to README

@anmarchenko anmarchenko marked this pull request as ready for review August 17, 2023 13:46
@anmarchenko anmarchenko requested review from a team August 17, 2023 13:46
Add Readme instructions for minitest and config options explanations

Co-authored-by: Brave Hager <bhager@justworks.com>
@@ -94,4 +95,7 @@ task :ci do
declare "❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ jruby" => "bundle exec appraisal cucumber-6 rake spec:cucumber"
declare "❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ jruby" => "bundle exec appraisal cucumber-7 rake spec:cucumber"
# declare "❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ❌ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ jruby" => "bundle exec appraisal cucumber-8 rake spec:cucumber"

# Minitest
declare "❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ❌ 2.5 / ❌ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ jruby" => "bundle exec appraisal minitest-5 rake spec:minitest"
Copy link
Contributor

Choose a reason for hiding this comment

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

If we are supporting ruby 2.7+, I guess it could time to cleanup the unnecessary stuff prior to 2.7.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, the PR will come

@anmarchenko anmarchenko merged commit 218a6cb into main Aug 18, 2023
4 checks passed
@anmarchenko anmarchenko deleted the anmarchenko/minitest_support branch September 6, 2023 08:14
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