Bug #22494
Rubocop can fail when there is ruby inside node_modules/
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($/)
Associated revisions
History
#1
Updated by The Foreman Bot over 4 years ago
- Assignee set to Eric Helms
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/7181 added
#2
Updated by Justin Sherrill over 4 years ago
- Legacy Backlogs Release (now unused) set to 284
- Status changed from Ready For Testing to Closed
- Category set to Tests
Fixes #22494 - Ignore any ruby in node_modules