Skip to content

Run ERBLint on ViewComponent markup#5477

Merged
aduth merged 2 commits intomainfrom
aduth-erblint-components
Oct 7, 2021
Merged

Run ERBLint on ViewComponent markup#5477
aduth merged 2 commits intomainfrom
aduth-erblint-components

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Oct 7, 2021

Why: Because these ERB files live outside app/views, but should still be subject to linting.

aduth added 2 commits October 7, 2021 10:33
**Why**: Because these ERB files live outside app/views, but should still be subject to linting.
Seems expansion support may be OS/shell-specific
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aduth aduth merged commit e1e9da8 into main Oct 7, 2021
@aduth aduth deleted the aduth-erblint-components branch October 7, 2021 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants