Skip to content

Update erb_lint to 0.7.0#11610

Merged
matthinz merged 4 commits intomainfrom
matthinz/update-erb-lint
Dec 6, 2024
Merged

Update erb_lint to 0.7.0#11610
matthinz merged 4 commits intomainfrom
matthinz/update-erb-lint

Conversation

@matthinz
Copy link
Copy Markdown
Contributor

@matthinz matthinz commented Dec 6, 2024

erb_lint 0.5.0 was starting to emit warnings about Rubocop deprecations. Updating to 0.7.0 silences those warnings. It looks like they changed the preferred config file name and invocation method, so I've updated those as well to avoid additional warnings.

0.5.0 was spitting out warnings related to rubocop deprecations

[skip changelog]
erb_lint complains if you invoke it with `erblint`. This silences that warning.
Copy link
Copy Markdown
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

Nice, the previous inconsistency between erblint, erb-lint, and erb_lint always tripped me up, nice that they've consolidated 👍

Copy link
Copy Markdown
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.

silence, warnings

@matthinz matthinz merged commit 769cd47 into main Dec 6, 2024
@matthinz matthinz deleted the matthinz/update-erb-lint branch December 6, 2024 19:53
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.

3 participants