Merge pull request #74 from RodrigoMNardi/bug/github/retry/old_prs

CI Retry
FRRouting · Jun 6, 2024 · f2a8324 · f2a8324
2 parents 40b265b + b9d4fdc
commit f2a8324
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/github/build/retry.rb b/lib/github/build/retry.rb
@@ -45,7 +45,7 @@ def enqueued_stages
 
       def enqueued_failure_tests
         @check_suite.ci_jobs.where.not(status: :success).each do |ci_job|
-          next unless ci_job.stage.configuration.can_retry?
+          next if must_skip_retry?(ci_job)
 
           logger(Logger::WARN, "Enqueue CiJob: #{ci_job.inspect}")
           ci_job.enqueue(@github)
@@ -57,6 +57,10 @@ def enqueued_failure_tests
 
       private
 
+      def must_skip_retry?(ci_job)
+        ci_job.stage.nil? || !ci_job.stage.configuration.can_retry?
+      end
+
       def logger(severity, message)
         @loggers.each do |logger_object|
           logger_object.add(severity, message)