[pull] master from ruby:master#1050
Merged
Merged
Conversation
…tation
`"a: b"` must be processed as a string value (`a: b`) not a map
value (`{"a" => "b"}`).
ruby/rubygems@3c3dfe3168
…arch_nonascii` (#16359) Improve performance of String#scrub by skipping ascii characters after finding a multi byte character using search_nonascii. Create benchmark for String#scrub to show performance difference made by including the new ascii skips.
ruby/rubygems@fc2310629a Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
… lock When multiple `bundle install` processes run concurrently, a race condition can cause issues. The second process populates its `Gem::Specification.stubs` and `@installed_specs` caches before acquiring the ProcessLock. While waiting for the lock, the first process installs gems. After acquiring the lock, the second process uses its stale cache and may not see the newly installed gems. This fix clears the caches immediately after acquiring the process lock, ensuring that any gems installed by another process while waiting for the lock are properly detected. Similar to #8539 which addressed a related cache invalidation issue for the `bundle update` command. Fixes ruby/rubygems#8473 ruby/rubygems@114ad0741c Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…ar_cache `Source::Rubygems#specs` memoizes the merged Index in `@specs`, so nilling only `@installed_specs` and `@default_specs` leaves a stale merged view alive after the process lock is re-acquired. Reset `@specs` and `@cached_specs` as well, matching what `local!` / `remote!` / `cached!` already do. ruby/rubygems@21a4cc4178
…ar peeking Replace `instance_variable_get` assertions with behavior-driven checks: object identity confirms memoization is invalidated for all four cached indexes, and a stubbed `Bundler.rubygems.installed_specs` verifies that fresh data is surfaced after the cache is cleared. ruby/rubygems@ab41874182
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )