From 6894a8d0c5184a308df9e3ee961a2322c91cff76 Mon Sep 17 00:00:00 2001 From: Justus Date: Wed, 22 Jan 2025 19:21:55 -0600 Subject: [PATCH] simplify logic in add_handled_dependencies to not use extra var --- updater/lib/dependabot/dependency_snapshot.rb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/updater/lib/dependabot/dependency_snapshot.rb b/updater/lib/dependabot/dependency_snapshot.rb index e74047343a..c92b5e4720 100644 --- a/updater/lib/dependabot/dependency_snapshot.rb +++ b/updater/lib/dependabot/dependency_snapshot.rb @@ -137,11 +137,10 @@ def mark_group_handled(group) sig { params(dependency_names: T.any(String, T::Array[String])).void } def add_handled_dependencies(dependency_names) assert_current_directory_set! - set = @handled_dependencies[@current_directory] || Set.new + @handled_dependencies[@current_directory] ||= Set.new names = Array(dependency_names) Dependabot.logger.info("Adding dependencies as handled: (#{names.join(', ')}).") - set += names - @handled_dependencies[@current_directory] = set + @handled_dependencies[@current_directory] += names end sig { returns(T::Set[String]) }