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 New feature or request ast AST Handling 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
@bflad bflad deleted the commentignore-multiline-comments branch January 26, 2022 00:37
bflad added a commit that referenced this pull request Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ast AST Handling enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for multiline comments
1 participant