Sign upLogin

fdutey/shirinji

View on GitHub
Star
lib/shirinji/map.rb

Summary

Maintainability
A
0 mins
Test Coverage

Use each_key instead of keys.each.
Open

      map.beans.keys.each { |name| raise_if_name_already_taken!(name) }
Severity: Minor
Found in lib/shirinji/map.rb by rubocop

This cop checks for uses of each_key and each_value Hash methods.

Note: If you have an array of two-element arrays, you can put parentheses around the block arguments to indicate that you're not working with a hash, and suppress RuboCop offenses.

Example:

# bad
hash.keys.each { |k| p k }
hash.values.each { |v| p v }
hash.each { |k, _v| p k }
hash.each { |_k, v| p v }

# good
hash.each_key { |k| p k }
hash.each_value { |v| p v }

The use of eval is a serious security risk.
Open

      eval(File.read(location))
Severity: Minor
Found in lib/shirinji/map.rb by rubocop

This cop checks for the use of Kernel#eval and Binding#eval.

Example:

# bad

eval(something)
binding.eval(something)

There are no issues that match your filters.

Category
Status