cloudamatic/mu

View on GitHub
cookbooks/mu-tools/recipes/eks.rb

Summary

Maintainability
A
0 mins
Test Coverage

Use meaningful heredoc delimiters.
Open

EOH
Severity: Minor
Found in cookbooks/mu-tools/recipes/eks.rb by rubocop

This cop checks that your heredocs are using meaningful delimiters. By default it disallows END and EO*, and can be configured through blacklisting additional delimiters.

Example:

# good
<<-SQL
  SELECT * FROM foo
SQL

# bad
<<-END
  SELECT * FROM foo
END

# bad
<<-EOS
  SELECT * FROM foo
EOS

Useless assignment to variable - max_pods.
Open

  max_pods = node['deployment']['container_clusters'][cluster_short_name]['max_pods']
Severity: Minor
Found in cookbooks/mu-tools/recipes/eks.rb by rubocop

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 - foo

Currently 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
end

Example:

# good

def some_method
  some_var = 1
  do_something(some_var)
end

Use meaningful heredoc delimiters.
Open

EOH
Severity: Minor
Found in cookbooks/mu-tools/recipes/eks.rb by rubocop

This cop checks that your heredocs are using meaningful delimiters. By default it disallows END and EO*, and can be configured through blacklisting additional delimiters.

Example:

# good
<<-SQL
  SELECT * FROM foo
SQL

# bad
<<-END
  SELECT * FROM foo
END

# bad
<<-EOS
  SELECT * FROM foo
EOS

There are no issues that match your filters.

Category
Status