Showing 51 of 51 total issues
unexpected token error Open
Open
%i(set add remove).each do |name|- Exclude checks
unexpected token tRPAREN Open
Open
def unmarshal(body, as:)- Exclude checks
cannot assign to a keyword Open
Open
def initialize(scheme, token: nil, params: {})- Exclude checks
module definition in method body Open
Open
module Xenon- Exclude checks
Ambiguous splat operator. Parenthesize the method arguments if it's surely a splat operator, or add a whitespace to the right of the * if it should be a multiplication. Open
Open
yield *values- Read upRead up
- Exclude checks
This cop checks for ambiguous operators in the first argument of a method invocation without parentheses.
Example:
# bad
# The `*` is interpreted as a splat operator but it could possibly be
# a `*` method invocation (i.e. `do_something.*(some_array)`).
do_something *some_arrayExample:
# good
# With parentheses, there's no ambiguity.
do_something(*some_array)Useless assignment to variable - value. Open
Open
value = convert_param_type(value, settings[:type]) if settings.has_key?(:type)- 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)
endclass definition in method body Open
Open
class GenericCredentials- Exclude checks
Ambiguous splat operator. Parenthesize the method arguments if it's surely a splat operator, or add a whitespace to the right of the * if it should be a multiplication. Open
Open
yield *values- Read upRead up
- Exclude checks
This cop checks for ambiguous operators in the first argument of a method invocation without parentheses.
Example:
# bad
# The `*` is interpreted as a splat operator but it could possibly be
# a `*` method invocation (i.e. `do_something.*(some_array)`).
do_something *some_arrayExample:
# good
# With parentheses, there's no ambiguity.
do_something(*some_array)TODO found Open
Open
# extras -- TODO: move these into header rules?- Exclude checks
TODO found Open
Open
tree = Xenon::Parsers.const_get("#{base_name}Header").new.parse(s) # TODO: HeaderParser would be better- Exclude checks
TODO found Open
Open
fail_with 500, error.message # TODO: Only if verbose errors configured- Exclude checks