Skip to content

Fallback to kicks gem for version check in SneakersSpy - #1655

Draft
vlado wants to merge 1 commit into
elastic:mainfrom
vlado:ruby_4_sneakers_kicks
Draft

Fallback to kicks gem for version check in SneakersSpy#1655
vlado wants to merge 1 commit into
elastic:mainfrom
vlado:ruby_4_sneakers_kicks

Conversation

@vlado

@vlado vlado commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

What does this pull request do?

After #1650 got merged we experienced issue with sneakers spy in our app.

Sneakers gem was abandoned and does not work with Ruby 4. It was replaced by the kicks gem.

Kicks gem still uses sneakers namespace so it will load SneakersSpy which fails because Gem.loaded_specs['sneakers'] returns nil. We fix it by adding fallback to Gem.loaded_specs['kicks'].
Last sneakers version is 2.12.0 and kicks starts from 3.0.0 so this should work properly.

Checklist

  • I have signed the Contributor License Agreement.
  • My code follows the style guidelines of this project (See .rubocop.yml)
  • I have rebased my changes on top of the latest main branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have made corresponding changes to the documentation
  • I have updated docs/release-notes/index.md
  • I have updated docs/reference/supported-technologies.md
  • Added an API method or config option? Document in which version this will be introduced

@vlado
vlado marked this pull request as draft July 28, 2026 15:49
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.

1 participant