Skip to content
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.

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

passes/commentignore: Support multiple line comments #249

Merged
merged 1 commit into from Jan 26, 2022

Conversation

bflad
Copy link
Owner

@bflad bflad commented Jan 25, 2022

Closes #248

Previously, multiple line comments including the lintignore directive would not take effect unless it was on the first line. Now any comment line can implement the lintignore directive.

Reference: #248

Previously, multiple line comments including the lintignore directive would not take effect unless it was on the first line. Now any comment line can implement the lintignore directive.
@bflad bflad added enhancement ast labels Jan 25, 2022
@bflad bflad added this to the v0.28.0 milestone Jan 25, 2022
@bflad bflad merged commit afaa086 into main Jan 26, 2022
6 checks passed
@bflad bflad deleted the commentignore-multiline-comments branch Jan 26, 2022
bflad added a commit that referenced this issue Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ast enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant