Skip to content

Update rubocop, brakeman, rspec, erb_lint, bullet, bootsnap gems#7569

Merged
mitchellhenke merged 8 commits intomainfrom
mitchellhenke/update-rubocop-part-3
Jan 3, 2023
Merged

Update rubocop, brakeman, rspec, erb_lint, bullet, bootsnap gems#7569
mitchellhenke merged 8 commits intomainfrom
mitchellhenke/update-rubocop-part-3

Conversation

@mitchellhenke
Copy link
Contributor

🛠 Summary of changes

Just a small update to use more recent gems

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

suggestion: 'Rename classes that are known to be hidden by the Hush plugin'
SpaceAroundErbTag:
enabled: true
CommentSyntax:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This adds a check to prevent incorrect comment tags in ERB (examples borrowed from the erb_lint repository):

<%# erb comment here %>
<%= # bad erb comment here %>
<%
  # apparently this comment syntax is valid?
%>
<% # very; bad erb comment here %>

@mitchellhenke mitchellhenke merged commit 0fb088b into main Jan 3, 2023
@mitchellhenke mitchellhenke deleted the mitchellhenke/update-rubocop-part-3 branch January 3, 2023 19:09
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