File git_repo.rb has 551 lines of code (exceeds 250 allowed). Consider refactoring.

Open

require "git"
require "tty-command"
require "securerandom"
require "digest"
require "task_queue"

Method setup_repo has a Cognitive Complexity of 45 (exceeds 5 allowed). Consider refactoring.

Open

    def setup_repo
      retry_count ||= 0
      if File.directory?(local_folder)
        # TODO: test if this crashes if it's not a git directory
        begin

• Read up
  Read up

Method run has 134 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def run(new_line_block:, completion_block:)
      if lane.nil?
        raise "Before calling `.run` on #{self}, you have to call `.setup` to finish preparing the BuildRunner"
      end

Method run has a Cognitive Complexity of 31 (exceeds 5 allowed). Consider refactoring.

Open

    def run(new_line_block:, completion_block:)
      if lane.nil?
        raise "Before calling `.run` on #{self}, you have to call `.setup` to finish preparing the BuildRunner"
      end

• Read up
  Read up

Class GitRepo has 31 methods (exceeds 20 allowed). Consider refactoring.

Open

  class GitRepo
    # rubocop:enable Metrics/ClassLength
    include FastlaneCI::Logging

    DEFAULT_REMOTE = "origin"

File build_runner.rb has 304 lines of code (exceeds 250 allowed). Consider refactoring.

Open

require_relative "../../shared/models/artifact"
require_relative "./build_runner_output_row"

# ideally we'd have a nicer way to inject this
require_relative "../../features/build/build_controller"

Class Services has 25 methods (exceeds 20 allowed). Consider refactoring.

Open

  class Services
    class << self
      include FastlaneCI::Logging
    end

Method initialize has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.

Open

    def initialize
      @mutex = Mutex.new
      @busy = false
      # TODO: do we need a thread here to do the work or does `scheduler.schedule` handle that?
      @thread = Thread.new do

• Read up
  Read up

File add-project-dialog.component.spec.ts has 272 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import {CommonModule} from '@angular/common';
import {DebugElement} from '@angular/core';
import {async, ComponentFixture, fakeAsync, TestBed} from '@angular/core/testing';
import {FormsModule, ReactiveFormsModule} from '@angular/forms';
import {MatButtonModule, MatDialogModule, MatDialogRef, MatIconModule, MatProgressSpinnerModule, MatSelectModule} from '@angular/material';

Class FastlaneCIOutput has 23 methods (exceeds 20 allowed). Consider refactoring.

Open

  class FastlaneCIOutput < FastlaneCore::Interface
    # The block being called for each new line that should
    # be stored in the log.
    attr_reader :each_line_block

Method setup_repo has 60 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def setup_repo
      retry_count ||= 0
      if File.directory?(local_folder)
        # TODO: test if this crashes if it's not a git directory
        begin

Class BuildRunner has 22 methods (exceeds 20 allowed). Consider refactoring.

Open

  class BuildRunner
    include FastlaneCI::Logging

    # Reference to the FastlaneCI::Project of this particular build run
    attr_reader :project

Method handle_exception has 57 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def handle_exception(ex, console_message: nil, exception_context: {})
      unless console_message.nil?
        logger.error(console_message)
      end
      logger.error(ex)

File project.component.spec.ts has 257 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import 'rxjs/add/operator/switchMap';

import {Location} from '@angular/common';
import {DebugElement} from '@angular/core';
import {async, ComponentFixture, fakeAsync, flush, TestBed, tick} from '@angular/core/testing';

Class RemoteRunner has 21 methods (exceeds 20 allowed). Consider refactoring.

Open

  class RemoteRunner
    include FastlaneCI::Logging

    # this record separator is used to delimit the history file.
    # if for whatever reason, we can't delimit on newlines, consider \30 the Record Separator char.

File build.component.spec.ts has 256 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import {DebugElement, Component} from '@angular/core';
import {Location} from '@angular/common';
import {async, ComponentFixture, TestBed, tick, fakeAsync} from '@angular/core/testing';
import {MatCardModule, MatProgressSpinnerModule} from '@angular/material';
import {By} from '@angular/platform-browser';

Method setup_tooling_environment? has 53 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def setup_tooling_environment?
      xcode_version_file_path = File.join(repo.local_folder, ".xcode-version")
      unless File.exist?(xcode_version_file_path)
        logger.debug("No `.xcode-version` file found for repo '#{project.project_name}', " \
                     "not managing Xcode for this one...")

Method commit_changes! has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

    def commit_changes!(commit_message: nil, push_after_commit: true, files_to_commit: [], repo_auth: self.repo_auth)
      git_action_with_queue do
        logger.debug("Starting commit_changes! #{git_config.git_url} for #{repo_auth.username}")
        commit_message ||= "Automatic commit by fastlane.ci"

• Read up
  Read up

Method start_workers_for_project_and_credential has 49 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    def start_workers_for_project_and_credential(project: nil, provider_credential: nil, notification_service:)
      user_responsible = provider_credential.ci_user

      if user_responsible.nil?
        name = project.project_name

Method enqueue_builds_for_open_github_prs_with_no_status has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

    def self.enqueue_builds_for_open_github_prs_with_no_status(projects: nil, github_service: nil)
      logger.debug("Searching for open PRs with no status and starting a build for them")
      projects.each do |project|
        # Don't enqueue builds for the open pull requests if we don't have a pull request trigger defined for it
        next if project.find_triggers_of_type(trigger_type: :pull_request).first.nil?

• Read up
  Read up