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.

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

nolint sub linter #4050

Closed
yonderblue opened this issue Aug 29, 2023 · 3 comments
Closed

nolint sub linter #4050

yonderblue opened this issue Aug 29, 2023 · 3 comments
Labels
duplicate This issue or pull request already exists

Comments

@yonderblue
Copy link

Your feature request related to a problem? Please describe.

Cannot do nolint:revive:cyclomatic or something like that

Describe the solution you'd like.

see above

Describe alternatives you've considered.

ignoring too much

Additional context.

No response
@yonderblue yonderblue added the enhancement New feature or improvement label Aug 29, 2023
@ldez
Copy link
Member

ldez commented Aug 29, 2023 •
edited
Loading

Duplicate of #1658 and #2706

@ldez ldez marked this as a duplicate of #1658 Aug 29, 2023
@ldez ldez closed this as completed Aug 29, 2023
@ldez ldez added duplicate This issue or pull request already exists and removed enhancement New feature or improvement labels Aug 29, 2023
@yonderblue
Copy link
Author

Does not seem to be a duplicate? I'm referring to something like mgechev/revive#885 (comment).

@ldez
Copy link
Member

ldez commented Aug 30, 2023

I already understood what you were proposing but it's a point that I talk about here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yonderblue @ldez