Skip to content

chore: cleanup gemfiles#2182

Merged
arielvalentin merged 2 commits intoopen-telemetry:mainfrom
thompson-tomo:logger_in_gemfiles
Apr 9, 2026
Merged

chore: cleanup gemfiles#2182
arielvalentin merged 2 commits intoopen-telemetry:mainfrom
thompson-tomo:logger_in_gemfiles

Conversation

@thompson-tomo
Copy link
Copy Markdown
Contributor

This removes the duplicate references to the logger gem in the gemfiles which are currently generating warnings.

@arielvalentin
Copy link
Copy Markdown
Contributor

I added these because of a warning that it would get removed from gems. Does this mean something else is requiring logger now?

@thompson-tomo
Copy link
Copy Markdown
Contributor Author

Logger has ended up being added twice to the gemfiles. Most of the time it is 2 lines higher/lower.

@arielvalentin arielvalentin added this pull request to the merge queue Apr 9, 2026
Merged via the queue into open-telemetry:main with commit 8f984fe Apr 9, 2026
76 of 77 checks passed
@thompson-tomo thompson-tomo deleted the logger_in_gemfiles branch April 9, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants