gitlabhq/gitlabhq

View on GitHub

Showing 1,237 of 1,706 total issues

File repository.rb has 1858 lines of code (exceeds 250 allowed). Consider refactoring.
Open

require 'tempfile'
require 'forwardable'
require "rubygems/package"

module Gitlab
Severity: Major
Found in lib/gitlab/git/repository.rb - About 5 days to fix

    Class Project has 246 methods (exceeds 20 allowed). Consider refactoring.
    Open

    class Project < ActiveRecord::Base
      include Gitlab::ConfigHelper
      include Gitlab::ShellAdapter
      include Gitlab::VisibilityLevel
      include AccessRequestable
    Severity: Major
    Found in app/models/project.rb - About 4 days to fix

      File project.rb has 1587 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      require 'carrierwave/orm/activerecord'
      
      class Project < ActiveRecord::Base
        include Gitlab::ConfigHelper
        include Gitlab::ShellAdapter
      Severity: Major
      Found in app/models/project.rb - About 4 days to fix

        Class Repository has 211 methods (exceeds 20 allowed). Consider refactoring.
        Open

            class Repository
              include Gitlab::Git::RepositoryMirroring
              include Gitlab::Git::Popen
              include Gitlab::EncodingHelper
              include Gitlab::Utils::StrongMemoize
        Severity: Major
        Found in lib/gitlab/git/repository.rb - About 4 days to fix

          File notes.js has 1487 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          /* eslint-disable no-restricted-properties, func-names, space-before-function-paren,
          no-var, prefer-rest-params, wrap-iife, no-use-before-define, camelcase,
          no-unused-expressions, quotes, max-len, one-var, one-var-declaration-per-line,
          default-case, prefer-template, consistent-return, no-alert, no-return-assign,
          no-param-reassign, prefer-arrow-callback, no-else-return, comma-dangle, no-new,
          Severity: Major
          Found in app/assets/javascripts/notes.js - About 3 days to fix

            Class User has 167 methods (exceeds 20 allowed). Consider refactoring.
            Open

            class User < ActiveRecord::Base
              extend Gitlab::ConfigHelper
            
              include Gitlab::ConfigHelper
              include Gitlab::SQL::Pattern
            Severity: Major
            Found in app/models/user.rb - About 3 days to fix

              File entities.rb has 1090 lines of code (exceeds 250 allowed). Consider refactoring.
              Open

              module API
                module Entities
                  class WikiPageBasic < Grape::Entity
                    expose :format
                    expose :slug
              Severity: Major
              Found in lib/api/entities.rb - About 2 days to fix

                Function UsersSelect has 525 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                function UsersSelect(currentUser, els, options = {}) {
                  var $els;
                  this.users = this.users.bind(this);
                  this.user = this.user.bind(this);
                  this.usersPath = "/autocomplete/users.json";
                Severity: Major
                Found in app/assets/javascripts/users_select.js - About 2 days to fix

                  Class MergeRequest has 128 methods (exceeds 20 allowed). Consider refactoring.
                  Open

                  class MergeRequest < ActiveRecord::Base
                    include AtomicInternalId
                    include IidRoutes
                    include Issuable
                    include Noteable
                  Severity: Major
                  Found in app/models/merge_request.rb - About 2 days to fix

                    Class Repository has 127 methods (exceeds 20 allowed). Consider refactoring.
                    Open

                    class Repository
                      REF_MERGE_REQUEST = 'merge-requests'.freeze
                      REF_KEEP_AROUND = 'keep-around'.freeze
                      REF_ENVIRONMENTS = 'environments'.freeze
                      MAX_DIVERGING_COUNT = 1000
                    Severity: Major
                    Found in app/models/repository.rb - About 2 days to fix

                      File user.rb has 982 lines of code (exceeds 250 allowed). Consider refactoring.
                      Open

                      require 'carrierwave/orm/activerecord'
                      
                      class User < ActiveRecord::Base
                        extend Gitlab::ConfigHelper
                      
                      
                      Severity: Major
                      Found in app/models/user.rb - About 2 days to fix

                        File services.rb has 842 lines of code (exceeds 250 allowed). Consider refactoring.
                        Open

                        module API
                          class Services < Grape::API
                            CHAT_NOTIFICATION_SETTINGS = [
                              {
                                required: true,
                        Severity: Major
                        Found in lib/api/services.rb - About 2 days to fix

                          File merge_request.rb has 839 lines of code (exceeds 250 allowed). Consider refactoring.
                          Open

                          class MergeRequest < ActiveRecord::Base
                            include AtomicInternalId
                            include IidRoutes
                            include Issuable
                            include Noteable
                          Severity: Major
                          Found in app/models/merge_request.rb - About 2 days to fix

                            File gl_dropdown.js has 812 lines of code (exceeds 250 allowed). Consider refactoring.
                            Open

                            /* eslint-disable func-names, no-underscore-dangle, space-before-function-paren, no-var, one-var, one-var-declaration-per-line, prefer-rest-params, max-len, vars-on-top, wrap-iife, no-unused-vars, quotes, no-shadow, no-cond-assign, prefer-arrow-callback, no-return-assign, no-else-return, camelcase, comma-dangle, no-lonely-if, guard-for-in, no-restricted-syntax, consistent-return, prefer-template, no-param-reassign, no-loop-func, no-mixed-operators */
                            /* global fuzzaldrinPlus */
                            
                            import $ from 'jquery';
                            import _ from 'underscore';
                            Severity: Major
                            Found in app/assets/javascripts/gl_dropdown.js - About 1 day to fix

                              Function constructor has 358 lines of code (exceeds 25 allowed). Consider refactoring.
                              Open

                                constructor(els, options = {}) {
                                  var _this, $els;
                                  _this = this;
                              
                                  $els = $(els);
                              Severity: Major
                              Found in app/assets/javascripts/labels_select.js - About 1 day to fix

                                File repository.rb has 745 lines of code (exceeds 250 allowed). Consider refactoring.
                                Open

                                require 'securerandom'
                                
                                class Repository
                                  REF_MERGE_REQUEST = 'merge-requests'.freeze
                                  REF_KEEP_AROUND = 'keep-around'.freeze
                                Severity: Major
                                Found in app/models/repository.rb - About 1 day to fix

                                  Class Build has 90 methods (exceeds 20 allowed). Consider refactoring.
                                  Open

                                    class Build < CommitStatus
                                      prepend ArtifactMigratable
                                      include TokenAuthenticatable
                                      include AfterCommitQueue
                                      include ObjectStorage::BackgroundMove
                                  Severity: Major
                                  Found in app/models/ci/build.rb - About 1 day to fix

                                    Notes has 75 functions (exceeds 20 allowed). Consider refactoring.
                                    Open

                                    export default class Notes {
                                      static initialize(
                                        notes_url,
                                        note_ids,
                                        last_fetched_at,
                                    Severity: Major
                                    Found in app/assets/javascripts/notes.js - About 1 day to fix

                                      File users.rb has 617 lines of code (exceeds 250 allowed). Consider refactoring.
                                      Open

                                      module API
                                        class Users < Grape::API
                                          include PaginationParams
                                          include APIGuard
                                          include Helpers::CustomAttributes
                                      Severity: Major
                                      Found in lib/api/users.rb - About 1 day to fix

                                        File users_select.js has 595 lines of code (exceeds 250 allowed). Consider refactoring.
                                        Open

                                        /* eslint-disable func-names, space-before-function-paren, one-var, no-var, prefer-rest-params, wrap-iife, quotes, max-len, one-var-declaration-per-line, vars-on-top, prefer-arrow-callback, consistent-return, comma-dangle, object-shorthand, no-shadow, no-unused-vars, no-else-return, no-self-compare, prefer-template, no-unused-expressions, no-lonely-if, yoda, prefer-spread, no-void, camelcase, no-param-reassign */
                                        /* global Issuable */
                                        /* global emitSidebarEvent */
                                        
                                        import $ from 'jquery';
                                        Severity: Major
                                        Found in app/assets/javascripts/users_select.js - About 1 day to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language