From 060e28faeecdceb1b6dd8c8edda8a0cf6314b13f Mon Sep 17 00:00:00 2001 From: Justus Date: Thu, 30 Jan 2025 04:03:28 -0600 Subject: [PATCH] fix types --- updater/lib/dependabot/dependency_snapshot.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/updater/lib/dependabot/dependency_snapshot.rb b/updater/lib/dependabot/dependency_snapshot.rb index b2c18d887b..4e39b40434 100644 --- a/updater/lib/dependabot/dependency_snapshot.rb +++ b/updater/lib/dependabot/dependency_snapshot.rb @@ -119,7 +119,7 @@ def job_group @dependency_group_engine.find_group(name: T.must(job.dependency_group_to_refresh)) end - sig { params(group: Dependabot::DependencyGroup, excluding_dependencies: T::Hash[Symbol, T::Set[String]]).void } + sig { params(group: Dependabot::DependencyGroup, excluding_dependencies: T::Hash[String, T::Set[String]]).void } def mark_group_handled(group, excluding_dependencies = {}) Dependabot.logger.info("Marking group '#{group.name}' as handled.") @@ -134,10 +134,10 @@ def mark_group_handled(group, excluding_dependencies = {}) # also add dependencies that might be in the group, as a rebase would add them; # this avoids individual PR creation that immediately is superseded by a group PR supersede current_dependencies = group.dependencies.map(&:name).reject do |dep| - excluding_dependencies[directory].include?(dep) + excluding_dependencies[directory]&.include?(dep) end - add_handled_dependencies(current_dependencies.concat(dependencies_in_existing_prs.map do |dep| + add_handled_dependencies(current_dependencies.concat(dependencies_in_existing_prs.filter_map do |dep| dep["dependency-name"] end)) end