Showing 89 of 89 total issues
Useless assignment to variable - e. Open
Open
rescue NoMethodError => e- Read upRead up
- Exclude checks
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 - fooCurrently 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
endExample:
# good
def some_method
some_var = 1
do_something(some_var)
endRedundant use of Object#to_s in interpolation. Open
Open
"DRYRUN_DUMMY_VALUE: #{self.class.to_s}"- Read upRead up
- Exclude checks
This cop checks for string conversion in string interpolation, which is redundant.
Example:
# bad
"result is #{something.to_s}"Example:
# good
"result is #{something}"Unused method argument - global_options. If it's necessary, use _ or _global_options as an argument name to indicate that it won't be used. Open
Open
def state_from_pipe(state = {}, global_options = self.global_options)- Read upRead up
- Exclude checks
This cop checks for unused method arguments.
Example:
# bad
def some_method(used, unused, _unused_but_allowed)
puts used
endExample:
# good
def some_method(used, _unused, _unused_but_allowed)
puts used
endUseless assignment to variable - resp. Open
Open
resp = s3Request(:delete_object, run_params)- Read upRead up
- Exclude checks
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 - fooCurrently 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
endExample:
# good
def some_method
some_var = 1
do_something(some_var)
endFIXME found Open
Open
FIXME not implemented yet- Exclude checks
FIXME found Open
Open
new :S3, 'FIXME S3 not implemented yet'- Exclude checks
FIXME found Open
Open
# FIXME raise some more appropriate type of error- Exclude checks
FIXME found Open
Open
# FIXME i should do this with URI- Exclude checks
FIXME found Open
Open
new :CLOUDANT, 'FIXME Cloudant not implemented yet'- Exclude checks