-
Notifications
You must be signed in to change notification settings - Fork 267
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
Fix comment ignore with trailing comment not recognized #3295
Fix comment ignore with trailing comment not recognized #3295
Conversation
The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).
|
private/bufpkg/bufcheck/client.go
Outdated
// | ||
// Invalid: | ||
// // buf:lint:ignore SERVICE_SUFFIX, SERVICE_PASCAL_CASE | ||
// // buf:lint:ignore SERVICE_PASCAL_CASE, SERVICE_SUFFIX |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this saying that you can do comma-separated ignores? I don't think that's in scope
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated comment in f516a9c (it also said ignoring SERVICE_PASCAL_CASE and this rule only
on top)
CHANGELOG.md
Outdated
@@ -2,7 +2,8 @@ | |||
|
|||
## [Unreleased] | |||
|
|||
- No changes yet. | |||
- Fix issue with `buf lint` where comment ignores in the shape of `// buf:lint:ignore <RULE_ID> <extra comment>` | |||
are not recognized due to the extra comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
were not recognized
not are not recognized
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in 78c78e6
This fixes the problem where comment ignores with a trailing comment is not recognized as a comment ignore: