bbatsov/rubocop

View on GitHub

Showing 627 of 669 total issues

Method mark_assignments_as_referenced_in_loop has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
Open

      def mark_assignments_as_referenced_in_loop(node)
        referenced_variable_names_in_loop, assignment_nodes_in_loop = find_variables_in_loop(node)

        referenced_variable_names_in_loop.each do |name|
          variable = variable_table.find_variable(name)
Severity: Minor
Found in lib/rubocop/cop/variable_force.rb - About 45 mins to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Method cops_body has 6 arguments (exceeds 4 allowed). Consider refactoring.
Open

  def cops_body(cop, description, examples_objects, safety_objects, see_objects, pars) # rubocop:disable Metrics/AbcSize, Metrics/ParameterLists
Severity: Minor
Found in lib/rubocop/cops_documentation_generator.rb - About 45 mins to fix

    Method on_if has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_if(node)
              traverse_node(node.condition) do |asgn_node|
                next unless asgn_node.loc.operator
    
                rhs = asgn_node.to_a.last
    Severity: Minor
    Found in lib/rubocop/cop/lint/literal_assignment_in_condition.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_if has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_if(node)
              previous = []
              while node.if? || node.elsif?
                condition = node.condition
                add_offense(condition) if previous.include?(condition)
    Severity: Minor
    Found in lib/rubocop/cop/lint/duplicate_elsif_condition.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              return unless (receiver = node.receiver)
              return unless receiver.receiver&.const_type? && receiver.receiver.short_name == :Dir
              return unless GLOB_METHODS.include?(receiver.method_name)
              return if multiple_argument?(receiver)
    Severity: Minor
    Found in lib/rubocop/cop/lint/redundant_dir_glob_sort.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method same_conditions_node_different_branch? has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def same_conditions_node_different_branch?(variable, outer_local_variable)
              variable_node = variable_node(variable)
              return false unless node_or_its_ascendant_conditional?(variable_node)
    
              outer_local_variable_node =
    Severity: Minor
    Found in lib/rubocop/cop/lint/shadowing_outer_local_variable.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_block_pass has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_block_pass(node)
              return if target_ruby_version >= 3.0
              return unless method_require?(node)
              return unless unsorted_dir_pass?(node.parent)
    
    
    Severity: Minor
    Found in lib/rubocop/cop/lint/non_deterministic_require_order.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              return unless node.receiver
    
              expression = single_argument_dig?(node)
              return unless expression
    Severity: Minor
    Found in lib/rubocop/cop/style/single_argument_dig.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              erb_new_with_non_keyword_arguments(node) do |arguments|
                return if arguments.empty? || correct_arguments?(arguments)
    
                arguments[1..3].each_with_index do |argument, i|
    Severity: Minor
    Found in lib/rubocop/cop/lint/erb_new_arguments.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_csend has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_csend(node)
              unless assume_receiver_instance_exists?(node.receiver)
                return unless check?(node) && allowed_method?(node.method_name)
                return if respond_to_nil_specific_method?(node)
              end
    Severity: Minor
    Found in lib/rubocop/cop/lint/redundant_safe_navigation.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method message has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def message(node)
              if require_parentheses_when_complex?
                command = parenthesized?(node.condition) ? 'Only use' : 'Use'
                format(MSG_COMPLEX, command: command)
              else
    Severity: Minor
    Found in lib/rubocop/cop/style/ternary_parentheses.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method autocorrect has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def autocorrect(corrector, node, condition, replacement, guard)
              corrector.replace(node.loc.keyword.join(condition.source_range), replacement)
    
              if_branch = node.if_branch
              else_branch = node.else_branch
    Severity: Minor
    Found in lib/rubocop/cop/style/guard_clause.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              return unless !node.arguments? && node.parenthesized?
              return if ineligible_node?(node)
              return if default_argument?(node)
              return if allowed_method_name?(node.method_name)
    Severity: Minor
    Found in lib/rubocop/cop/style/method_call_without_args_parentheses.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method handle_conversion_method has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def handle_conversion_method(node)
              to_method(node) do |receiver, to_method|
                next if receiver.nil? || allow_receiver?(receiver)
    
                message = format(
    Severity: Minor
    Found in lib/rubocop/cop/lint/number_conversion.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method process_multiple_assignment has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

              def process_multiple_assignment(masgn_node)
                mlhs_node, mrhs_node = *masgn_node
    
                mlhs_node.children.each_with_index do |lhs_node, index|
                  next unless ASSIGNMENT_TYPES.include?(lhs_node.type)
    Severity: Minor
    Found in lib/rubocop/cop/lint/useless_setter_call.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method check_var has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def check_var(node)
              return unless node.variable? || node.const_type?
    
              if node.const_type?
                template = node.special_keyword? ? VAR_MSG : CONST_MSG
    Severity: Minor
    Found in lib/rubocop/cop/lint/void.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              return unless node.receiver
    
              if offense?(node)
                prefer = prefer(node)
    Severity: Minor
    Found in lib/rubocop/cop/style/lambda_call.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_send has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_send(node)
              return unless deprecated_class_method?(node)
    
              offense_range = offense_range(node)
              prefer = preferred_method(node)
    Severity: Minor
    Found in lib/rubocop/cop/lint/deprecated_class_methods.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method found_attr has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def found_attr(node, args, readable: false, writable: false)
              args.each do |arg|
                name = sym_name(arg)
                next unless name
    
    
    Severity: Minor
    Found in lib/rubocop/cop/lint/duplicate_methods.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method on_percent_literal has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
    Open

            def on_percent_literal(node)
              return unless contains_quotes_or_commas?(node)
    
              add_offense(node) do |corrector|
                node.each_value do |value|
    Severity: Minor
    Found in lib/rubocop/cop/lint/percent_string_array.rb - About 45 mins to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Severity
    Category
    Status
    Source
    Language