Sign upLogin

codebreakdown/rack-vendor_accept_header

View on GitHub
Star

Showing 12 of 12 total issues

Dependencies should be sorted in an alphabetical order within their section of the gemspec. Dependency rack-test should appear before rspec.
Open

  spec.add_development_dependency 'rack-test', '~> 0.6'
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Dependencies in the gemspec should be alphabetically sorted.

Example:

# bad
spec.add_dependency 'rubocop'
spec.add_dependency 'rspec'

# good
spec.add_dependency 'rspec'
spec.add_dependency 'rubocop'

# good
spec.add_dependency 'rubocop'

spec.add_dependency 'rspec'

# bad
spec.add_development_dependency 'rubocop'
spec.add_development_dependency 'rspec'

# good
spec.add_development_dependency 'rspec'
spec.add_development_dependency 'rubocop'

# good
spec.add_development_dependency 'rubocop'

spec.add_development_dependency 'rspec'

# bad
spec.add_runtime_dependency 'rubocop'
spec.add_runtime_dependency 'rspec'

# good
spec.add_runtime_dependency 'rspec'
spec.add_runtime_dependency 'rubocop'

# good
spec.add_runtime_dependency 'rubocop'

spec.add_runtime_dependency 'rspec'

# good only if TreatCommentsAsGroupSeparators is true
# For code quality
spec.add_dependency 'rubocop'
# For tests
spec.add_dependency 'rspec'

Freeze mutable objects assigned to constants.
Open

    VERSION = '1.0.0'
Severity: Minor
Found in lib/rack/vendor_accept_header/version.rb by rubocop

This cop checks whether some constant value isn't a mutable literal (e.g. array or hash).

Example:

# bad
CONST = [1, 2, 3]

# good
CONST = [1, 2, 3].freeze

Use %q only for strings that contain both single quotes and double quotes.
Open

  spec.description   = %q{Rack middleware to aid with parsing vendor accept headers}
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Line is too long. [84/80]
Open

  spec.description   = %q{Rack middleware to aid with parsing vendor accept headers}
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Line is too long. [104/80]
Open

  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression.
Open

      if match
Severity: Minor
Found in lib/rack/vendor_accept_header.rb by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression

Example:

# bad
def test
  if something
    work
  end
end

# good
def test
  return unless something
  work
end

# also good
def test
  work if something
end

# bad
if something
  raise 'exception'
else
  ok
end

# good
raise 'exception' if something
ok

Dependencies should be sorted in an alphabetical order within their section of the gemspec. Dependency codeclimate-test-reporter should appear before simplecov.
Open

  spec.add_development_dependency 'codeclimate-test-reporter', '~> 0.4'
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Dependencies in the gemspec should be alphabetically sorted.

Example:

# bad
spec.add_dependency 'rubocop'
spec.add_dependency 'rspec'

# good
spec.add_dependency 'rspec'
spec.add_dependency 'rubocop'

# good
spec.add_dependency 'rubocop'

spec.add_dependency 'rspec'

# bad
spec.add_development_dependency 'rubocop'
spec.add_development_dependency 'rspec'

# good
spec.add_development_dependency 'rspec'
spec.add_development_dependency 'rubocop'

# good
spec.add_development_dependency 'rubocop'

spec.add_development_dependency 'rspec'

# bad
spec.add_runtime_dependency 'rubocop'
spec.add_runtime_dependency 'rspec'

# good
spec.add_runtime_dependency 'rspec'
spec.add_runtime_dependency 'rubocop'

# good
spec.add_runtime_dependency 'rubocop'

spec.add_runtime_dependency 'rspec'

# good only if TreatCommentsAsGroupSeparators is true
# For code quality
spec.add_dependency 'rubocop'
# For tests
spec.add_dependency 'rspec'

Add an empty line after magic comments.
Open

lib = File.expand_path('../lib', __FILE__)
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Checks for a newline after the final magic comment.

Example:

# good
# frozen_string_literal: true

# Some documentation for Person
class Person
  # Some code
end

# bad
# frozen_string_literal: true
# Some documentation for Person
class Person
  # Some code
end

Unnecessary utf-8 encoding comment.
Open

# coding: utf-8
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

Use %q only for strings that contain both single quotes and double quotes.
Open

  spec.summary       = %q{Rack middleware to parse vendor accept headers}
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

%q-literals should be delimited by ( and ).
Open

  spec.summary       = %q{Rack middleware to parse vendor accept headers}
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

This cop enforces the consistent usage of %-literal delimiters.

Specify the 'default' key to set all preferred delimiters at once. You can continue to specify individual preferred delimiters to override the default.

Example:

# Style/PercentLiteralDelimiters:
#   PreferredDelimiters:
#     default: '[]'
#     '%i':    '()'

# good
%w[alpha beta] + %i(gamma delta)

# bad
%W(alpha #{beta})

# bad
%I(alpha beta)

%q-literals should be delimited by ( and ).
Open

  spec.description   = %q{Rack middleware to aid with parsing vendor accept headers}
Severity: Minor
Found in rack-vendor_accept_header.gemspec by rubocop

This cop enforces the consistent usage of %-literal delimiters.

Specify the 'default' key to set all preferred delimiters at once. You can continue to specify individual preferred delimiters to override the default.

Example:

# Style/PercentLiteralDelimiters:
#   PreferredDelimiters:
#     default: '[]'
#     '%i':    '()'

# good
%w[alpha beta] + %i(gamma delta)

# bad
%W(alpha #{beta})

# bad
%I(alpha beta)
Severity
Category
Status
Source
Language