Sign upLogin

orange-cloudfoundry/cf-ops-automation

View on GitHub
Star
lib/cf_apps.rb

Summary

Maintainability
A
35 mins
Test Coverage
A
100%

Method load_cf_app_details has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

  def load_cf_app_details(cf_app_name, cf_app_details, file, dir, cf_apps)
Severity: Minor
Found in lib/cf_apps.rb - About 35 mins to fix

CfApps has no descriptive comment
Open

class CfApps
Severity: Minor
Found in lib/cf_apps.rb by reek

Classes and modules are the units of reuse and release. It is therefore considered good practice to annotate every class and module with a brief comment outlining its responsibilities.

Example

Given

class Dummy
  # Do things...
end

Reek would emit the following warning:

test.rb -- 1 warning:
  [1]:Dummy has no descriptive comment (IrresponsibleModule)

Fixing this is simple - just an explaining comment:

# The Dummy class is responsible for ...
class Dummy
  # Do things...
end

There are no issues that match your filters.

Category
Status