dry-rb/dry-matcher

View on GitHub
lib/dry/matcher/evaluator.rb

Summary

Maintainability
A
0 mins
Test Coverage
A
95%

Use a guard clause instead of wrapping the code inside a conditional expression.
Open

        unless defined? @output
Severity: Minor
Found in lib/dry/matcher/evaluator.rb by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression

Example:

# bad
def test
  if something
    work
  end
end

# good
def test
  return unless something
  work
end

# also good
def test
  work if something
end

# bad
if something
  raise 'exception'
else
  ok
end

# good
raise 'exception' if something
ok

Use a guard clause instead of wrapping the code inside a conditional expression.
Open

        if @unhandled_cases.any?
Severity: Minor
Found in lib/dry/matcher/evaluator.rb by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression

Example:

# bad
def test
  if something
    work
  end
end

# good
def test
  return unless something
  work
end

# also good
def test
  work if something
end

# bad
if something
  raise 'exception'
else
  ok
end

# good
raise 'exception' if something
ok

Line is too long. [102/100]
Open

          ::Kernel.raise NonExhaustiveMatchError, "cases +#{@unhandled_cases.join(', ')}+ not handled"
Severity: Minor
Found in lib/dry/matcher/evaluator.rb by rubocop

There are no issues that match your filters.

Category
Status