Growstuff/growstuff

View on GitHub
app/views/plantings/_form.html.haml

Summary

Maintainability
Test Coverage

Expected an indentation at 22 instead of at 21.
Invalid

                     value: @planting.planted_at ? @planting.planted_at.to_fs(:ymd) : '',

Expected an indentation at 22 instead of at 21.
Open

                     placeholder: 'optional'

Expected an indentation at 22 instead of at 21.
Open

                     class: 'add-datepicker', label: 'When?'

Expected an indentation at 22 instead of at 21.
Open

                     class: 'add-datepicker',

Expected an indentation at 22 instead of at 21.
Invalid

                     value: @planting.finished_at ? @planting.finished_at.to_fs(:ymd) : '',

Expected an indentation at 22 instead of at 21.
Open

                     label: 'Finished date',

The = symbol should have one space separating it from code
Open

        %strong=yield :title

HamlLint/SpaceBeforeScript

Separate Ruby script indicators (-/=) from their code with a single space.

Bad: no space between = and some_expression haml =some_expression

Good haml = some_expression

Good haml - some_value = 'Hello World'

Ensuring space after -/= enforces a consistency that all HAML tags/script indicators are separated from their inline content by a space. Since it is optional to add a space after -/= but required when writing %tag or similar, the consistency is best enforced via a linter.

Line is too long. [126/120]
Open

            Tip: Plan our your future plantings by forward dating, and subscribe to your iCalendar feed for reminders to plant

HamlLint/LineLength

Option Description
max Maximum number of columns a single line can have. (default 80)

Wrap lines at 80 characters. You can configure this amount via the max option on the linter, e.g. by adding the following to your .haml-lint.yml:

linters:
  LineLength:
    max: 100

Long lines are harder to read and usually indicative of complexity. You can avoid them by splitting long attribute hashes on a comma, for example:

%tag{ attr1: 1,
      attr2: 2,
      attr3: 3 }

This significantly improves readability.

There are no issues that match your filters.

Category
Status