firehoseio/firehose

View on GitHub
lib/firehose/server/subscriber.rb

Summary

Maintainability
A
0 mins
Test Coverage
A
96%

TODO found
Open

        # TODO: Instead of just raising an exception, it would probably be better
Severity: Minor
Found in lib/firehose/server/subscriber.rb by fixme

Use == if you meant to do a comparison or wrap the expression in parentheses to indicate you meant to assign in a condition.
Open

          if channels = subscriptions.delete(channel_key)
Severity: Minor
Found in lib/firehose/server/subscriber.rb by rubocop

This cop checks for assignments in the conditions of if/while/until.

Example:

# bad

if some_var = true
  do_something
end

Example:

# good

if some_var == true
  do_something
end

There are no issues that match your filters.

Category
Status