Project

General

Profile

Actions

Bug #22494

closed

Rubocop can fail when there is ruby inside node_modules/

Added by Eric Helms about 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description


/var/lib/workspace/workspace/katello-pr-test-test/node_modules/patternfly/node_modules/font-awesome-sass/lib/font_awesome/sass/rails/railtie.rb:9:9: W: Lint/BlockAlignment: end at 9, 8 is not aligned with initializer "font-awesome-sass.view_helpers" do at 7, 7.

        end

        ^^^

/var/lib/workspace/workspace/katello-pr-test-test/node_modules/patternfly/node_modules/font-awesome-sass/lib/font_awesome/sass/version.rb:3:15: C: Style/MutableConstant: Freeze mutable objects assigned to constants.

    VERSION = '4.7.0'

              ^^^^^^^

/var/lib/workspace/workspace/katello-pr-test-test/node_modules/patternfly/node_modules/font-awesome-sass/font-awesome-sass.gemspec:2:1: C: Layout/EmptyLineAfterMagicComment: Add an empty line after magic comments.

lib = File.expand_path('../lib', __FILE__)

^

/var/lib/workspace/workspace/katello-pr-test-test/node_modules/patternfly/node_modules/font-awesome-sass/font-awesome-sass.gemspec:16:45: C: Style/SpecialGlobalVars: Prefer $INPUT_RECORD_SEPARATOR or $RS from the stdlib 'English' module (don't forget to require it) over $/.

  spec.files         = `git ls-files`.split($/)
Actions #1

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Eric Helms
  • Pull request https://github.com/Katello/katello/pull/7181 added
Actions #2

Updated by Justin Sherrill about 6 years ago

  • Category set to Tests
  • Status changed from Ready For Testing to Closed
  • translation missing: en.field_release set to 284
Actions

Also available in: Atom PDF