Skip to content

Linter: Rename erb-require-trailing-newline linter rule #660

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
marcoroth merged 1 commit into from
Oct 16, 2025
Merged

Linter: Rename erb-require-trailing-newline linter rule #660

marcoroth merged 1 commit into from
Oct 16, 2025

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Oct 15, 2025

This pull request renames the erb-requires-trailing-newline linter rule to erb-require-trailing-newline to be more in line with the other erb-require-whitespace-inside-tags linter rule name.
@marcoroth marcoroth marked this pull request as ready for review October 15, 2025 22:13
@marcoroth marcoroth force-pushed the linter-rename-erb-require-trailing-newline branch from 960f297 to ae4a108 Compare October 15, 2025 22:39
@marcoroth marcoroth merged commit 494764c into main Oct 16, 2025
7 checks passed
@marcoroth marcoroth deleted the linter-rename-erb-require-trailing-newline branch October 16, 2025 09:57
@asilano asilano mentioned this pull request Oct 21, 2025
Merged
asilano pushed a commit to fac/herb that referenced this pull request Oct 21, 2025
@marcoroth @asilano
Linter: Rename erb-require-trailing-newline linter rule (marcoroth#660
d5afc84 
)

This pull request renames the `erb-requires-trailing-newline` linter
rule to `erb-require-trailing-newline` to be more in line with the other
`erb-require-whitespace-inside-tags` linter rule name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linter linter-rule refactor

2 participants

@marcoroth