Skip to content

chore: Replicate simplecov changes to config gems - #2342

Merged
mwear merged 1 commit into
open-telemetry:mainfrom
thompson-tomo:replicate-simplecov
Sep 14, 2026
Merged

mwear merged 1 commit into
open-telemetry:mainfrom
thompson-tomo:replicate-simplecov

Conversation

@thompson-tomo

@thompson-tomo thompson-tomo commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

This replicates the changes in #2117 to the config gem as it was missed

@thompson-tomo thompson-tomo changed the title Replicate simplecov changes chore: Replicate simplecov changes Sep 8, 2026
@thompson-tomo
thompson-tomo marked this pull request as ready for review September 8, 2026 14:30
@thompson-tomo thompson-tomo changed the title chore: Replicate simplecov changes chore: Replicate recent changes in missed gems Sep 8, 2026
@thompson-tomo thompson-tomo changed the title chore: Replicate recent changes in missed gems chore: Replicate simplecov changes to config gems Sep 8, 2026
Comment thread config/Gemfile
gemspec

group :test, :development do
gem 'dotenv', '3.2.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.

This is in already evaled root Gemfile 🤔.

gem 'dotenv', '~> 3.2.0'

@thompson-tomo thompson-tomo Sep 9, 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.

You are correct but I would rather have it as it is to be consistent with all the other gems & we can tackle further optimisations in a future pr.

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.

Doesn't that emit warning (or failure) in bundler?

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.

Neither, it is just an info

2026-09-08T14:53:01.2947238Z Your Gemfile lists the gem dotenv (~> 3.2.0) more than once.
2026-09-08T14:53:01.2948161Z You should probably keep only one of them.
2026-09-08T14:53:01.2948774Z Remove any duplicate entries and specify the gem only once.
2026-09-08T14:53:01.2949671Z While it's not a problem now, it could cause errors if you change the version of one of them later.

We have renovate which will update both of them hence we would avoid the error scenario

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 hear @simi's concern, but agree with @thompson-tomo that we should address it in a different PR. @thompson-tomo would you mind opening an issue to address this?

@thompson-tomo thompson-tomo Sep 10, 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.

Done #2391

@mwear
mwear merged commit b10d089 into open-telemetry:main Sep 14, 2026
66 of 67 checks passed
@thompson-tomo
thompson-tomo deleted the replicate-simplecov branch September 15, 2026 01:59
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