Sign upLogin

clayton/consummo

View on GitHub
Star

Showing 3 of 3 total issues

Useless assignment to variable - e.
Open

      rescue Exception => e
Severity: Minor
Found in lib/consummo/enrichers/social_media_metric_enricher.rb by rubocop

This cop checks for every useless assignment to local variable in every scope. The basic idea for this cop was from the warning of ruby -cw:

assigned but unused variable - foo

Currently this cop has advanced logic that detects unreferenced reassignments and properly handles varied cases such as branch, loop, rescue, ensure, etc.

Example:

# bad

def some_method
  some_var = 1
  do_something
end

Example:

# good

def some_method
  some_var = 1
  do_something(some_var)
end

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

    def enrich(item)
Severity: Minor
Found in lib/consummo/enrichers/simple_enricher.rb by rubocop

This cop checks for unused method arguments.

Example:

# bad

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

Example:

# good

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

Avoid rescuing the Exception class. Perhaps you meant to rescue StandardError?
Open

      rescue Exception => e
        {}
Severity: Minor
Found in lib/consummo/enrichers/social_media_metric_enricher.rb by rubocop

This cop checks for rescue blocks targeting the Exception class.

Example:

# bad

begin
  do_something
rescue Exception
  handle_exception
end

Example:

# good

begin
  do_something
rescue ArgumentError
  handle_exception
end
Severity
Category
Status
Source
Language