ManageIQ/manageiq

View on GitHub
tools/log_processing/perf_timings.rb

Summary

Maintainability
A
0 mins
Test Coverage

Avoid immutable Array literals in loops. It is better to extract it into a local variable or a constant.
Open

    timings.delete_if { |k, _v| [:start_range, :end_range, :num_vim_queries, :num_vim_trips, :collect_metrics].include?(k) }

The use of eval is a serious security risk.
Open

  timings = eval(timings)

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