ManageIQ/manageiq-providers-vmware

View on GitHub
tools/event_history.rb

Summary

Maintainability
A
0 mins
Test Coverage

Unused method argument - argv. If it's necessary, use _ or _argv as an argument name to indicate that it won't be used. If it's unnecessary, remove it. You can also write as parse_args(*) if you want the method to accept any arguments but don't care about them.
Open

def parse_args(argv)
Severity: Minor
Found in tools/event_history.rb by rubocop

Checks for unused method arguments.

Example:

# bad
def some_method(used, unused, _unused_but_allowed)
  puts used
end

# good
def some_method(used, _unused, _unused_but_allowed)
  puts used
end

Example: AllowUnusedKeywordArguments: false (default)

# bad
def do_something(used, unused: 42)
  used
end

Example: AllowUnusedKeywordArguments: true

# good
def do_something(used, unused: 42)
  used
end

Example: IgnoreEmptyMethods: true (default)

# good
def do_something(unused)
end

Example: IgnoreEmptyMethods: false

# bad
def do_something(unused)
end

Example: IgnoreNotImplementedMethods: true (default)

# good
def do_something(unused)
  raise NotImplementedError
end

def do_something_else(unused)
  fail "TODO"
end

Example: IgnoreNotImplementedMethods: false

# bad
def do_something(unused)
  raise NotImplementedError
end

def do_something_else(unused)
  fail "TODO"
end

There are no issues that match your filters.

Category
Status