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 all_specs deprecation of bundler #1159

Merged
merged 4 commits into from
Jul 5, 2024

Conversation

jbockler
Copy link
Contributor

@jbockler jbockler commented Jul 5, 2024

bundler deprecated the usage of Bundler.rubygems.all_specs with version 2.5.12 (June 13, 2024), in order to better support default gems.
This version is the new default version with ruby 3.3.3 and thus the warning pops up:

Bundler.rubygems.all_specs has been removed in favor of Bundler.rubygems.installed_specs

bundler deprecated the usage of `Bundler.rubygems.all_specs`
with version 2.5.12 (June 13, 2024), in order to better
support default gems.
This version is the new default version with ruby 3.3.3
and thus the warning pops up:

Bundler.rubygems.all_specs has been removed in favor
of Bundler.rubygems.installed_specs
Copy link
Member

@tombruijn tombruijn left a comment

Choose a reason for hiding this comment

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

Great! Thanks for the PR!

@tombruijn tombruijn self-assigned this Jul 5, 2024
@tombruijn tombruijn added the chore label Jul 5, 2024
@tombruijn tombruijn merged commit 4d8299a into appsignal:main Jul 5, 2024
117 checks passed
@jbockler jbockler deleted the bundler-deprecation branch July 5, 2024 13:59
@jbockler
Copy link
Contributor Author

jbockler commented Jul 5, 2024

Thanks for the quick merge and small style fixes!

tombruijn added a commit that referenced this pull request Aug 26, 2024
Follow up of #1159

Silence the warning about calling an deprecated method because it
confuses me when I read the spec output.

Test against the new method as well.
tombruijn added a commit that referenced this pull request Aug 26, 2024
Follow up of #1159

Silence the warning about calling an deprecated method because it
confuses me when I read the spec output.

Test against the new method as well.
tombruijn added a commit that referenced this pull request Aug 26, 2024
Follow up of #1159

Silence the warning about calling an deprecated method because it
confuses me when I read the spec output.

Test against the new method as well.
tombruijn added a commit that referenced this pull request Aug 26, 2024
Follow up of #1159

Silence the warning about calling an deprecated method because it
confuses me when I read the spec output.

Test against the new method as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants