Don't unnecessarily loop twice through dependencies

rubygems · Dec 1, 2021 · 06b4a79 · 06b4a79
1 parent f1d33fa
commit 06b4a79
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/bundler/lib/bundler/definition.rb b/bundler/lib/bundler/definition.rb
@@ -634,15 +634,13 @@ def converge_sources
     end
 
     def converge_dependencies
+      changes = false
+
       @dependencies.each do |dep|
         if dep.source
           dep.source = sources.get(dep.source)
         end
-      end
 
-      changes = false
-
-      @dependencies.each do |dep|
         unless locked_dep = @locked_deps[dep.name]
           changes = true
           next