Sign upLogin

sanger/sequencescape

View on GitHub
Star
lib/label_printer/label_printer/label/sample_manifest_redirect.rb

Summary

Maintainability
A
0 mins
Test Coverage
A
93%

LabelPrinter::Label::SampleManifestRedirect has no descriptive comment
Open

    class SampleManifestRedirect
Severity: Minor
Found in lib/label_printer/label_printer/label/sample_manifest_redirect.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

Complex method LabelPrinter::Label::SampleManifestRedirect#labels (21.5)
Open

      def labels # rubocop:todo Metrics/MethodLength
        case sample_manifest.asset_type
        when 'plate', 'library_plate'
          if @printer_type_class.double_label?
            SampleManifestPlateDouble.new(options).labels
Severity: Minor
Found in lib/label_printer/label_printer/label/sample_manifest_redirect.rb by flog

Flog calculates the ABC score for methods. The ABC score is based on assignments, branches (method calls), and conditions.

You can read more about ABC metrics or the flog tool

There are no issues that match your filters.

Category
Status