Skip to content

chore(deps): Upgrade dependency minitest to v6#2197

Merged
kaylareopelle merged 2 commits into
open-telemetry:mainfrom
thompson-tomo:minitest/v6
Jun 23, 2026
Merged

chore(deps): Upgrade dependency minitest to v6#2197
kaylareopelle merged 2 commits into
open-telemetry:mainfrom
thompson-tomo:minitest/v6

Conversation

@thompson-tomo

Copy link
Copy Markdown
Contributor

See #2180 for details.

This PR is a subset of #2180, scoped to the gems which can be updated without changes.

This reduces qty of gems out of date by ensuring they can be updated going forward and avoids the chance changes could be added which would block a v6 update.

Comment thread propagator/jaeger/Gemfile

group :test, :development do
gem 'minitest', '~> 5.27.0'
gem 'minitest', '~> 6.0.0'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm hesitant to continue updating the dependencies on Jaeger since the library itself is deprecated. Is there a way to exclude it from Renovate?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think it's fine for this change, but I'll open an issue.

@thompson-tomo thompson-tomo Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually the deprecated gem is the jaeger exporter not propagator. I have opened #2205 to exclude the deprecated from renovate & rubocop as per the instructions in open-telemetry/opentelemetry-ruby-contrib#2219

@kaylareopelle kaylareopelle requested review from a team as code owners June 23, 2026 16:20
@kaylareopelle kaylareopelle merged commit c0f34bb into open-telemetry:main Jun 23, 2026
63 checks passed
@thompson-tomo thompson-tomo deleted the minitest/v6 branch June 23, 2026 16:37
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