Skip to content

Commit

Permalink
[rubygems/rubygems] Rename method for clarity
Browse files Browse the repository at this point in the history
And also so that it matches the method used by main PubGrub sample
resolver class.

rubygems/rubygems@0e612361b8
  • Loading branch information
deivid-rodriguez authored and matzbot committed Apr 1, 2024
1 parent 3ca0683 commit d342937
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/bundler/resolver.rb
Expand Up @@ -158,7 +158,7 @@ def parse_dependency(package, dependency)
def versions_for(package, range=VersionRange.any)
versions = range.select_versions(@sorted_versions[package])

sort_versions(package, versions)
sort_versions_by_preferred(package, versions)
end

def no_versions_incompatibility_for(package, unsatisfied_term)
Expand Down Expand Up @@ -275,7 +275,7 @@ def all_versions_for(package)
groups
end

sort_versions(package, versions)
sort_versions_by_preferred(package, versions)
end

def source_for(name)
Expand Down Expand Up @@ -357,7 +357,7 @@ def requirement_satisfied_by?(requirement, spec)
requirement.satisfied_by?(spec.version) || spec.source.is_a?(Source::Gemspec)
end

def sort_versions(package, versions)
def sort_versions_by_preferred(package, versions)
if versions.size > 1
@gem_version_promoter.sort_versions(package, versions).reverse
else
Expand Down

0 comments on commit d342937

Please sign in to comment.