Bug #8675
closedrubocop does not ignore engines/bastion_katello/node_modules/ files
Description
es.rb:4:1: C: Extra empty line detected at body beginning.
engines/bastion_katello/node_modules/bastion/config/routes.rb:12:42: W: Unused block argument - name. If it's necessary, use _ or _name as an argument name to indicate that it won't be used.
pages = Bastion.plugins.collect { |name, plugin| plugin[:pages] }
^^^
engines/bastion_katello/node_modules/bastion/config/routes.rb:15:1: C: Extra empty line detected at body end.
engines/bastion_katello/node_modules/bastion/lib/bastion.rb:4:1: C: Favor if over unless for negative conditions.
require lib_foreman unless !Dir.exist?(lib_foreman)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
engines/bastion_katello/node_modules/bastion/lib/bastion.rb:12:1: C: Extra empty line detected at body beginning.
engines/bastion_katello/node_modules/bastion/lib/bastion.rb:13:3: C: Replace class var @plugins with a class instance var.
@plugins = {};
^^^^^^^
engines/bastion_katello/node_modules/bastion/lib/bastion.rb:13:17: C: Do not use semicolons to terminate expressions.
@@plugins = {};
^
engines/bastion_katello/node_modules/bastion/lib/bastion.rb:22:1: C: Extra empty line detected at body end.
engines/bastion_katello/node_modules/bastion/lib/bastion/engine.rb:5:1: C: Extra empty line detected at body beginning.
engines/bastion_katello/node_modules/bastion/lib/bastion/engine.rb:40:1: C: Extra empty line detected at body end.
Updated by The Foreman Bot about 10 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/4869 added
- Pull request deleted (
)
Updated by Justin Sherrill about 10 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|d34fd0643538bdac2ffd8a18ac8da2e7e147dec5.
Updated by Eric Helms almost 10 years ago
- Translation missing: en.field_release set to 23