-
Notifications
You must be signed in to change notification settings - Fork 3
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
NPP045 should be a Linter thing #347
Comments
Do you agree? @MarkusAmshove then I can do this and use this NPP045 for my new NPP050 instead. |
Yeah, I can move it into a linter rule |
I’ll do it. |
When you remove the rule/the check you should see the failing tests and make linter tests out of them :-) |
Not sure what you mean? Maybe you should do it after all? ;) |
The current tests for this rule are in a resources file and not in a Java file I think. But I can also do it if you want :-) |
Maybe it’s better if you do it. But no rush. :) |
Is #394 ready to merge? I'll do the number adjustment after merging then, so the builds stay green :-) |
In my opinion :)
The text was updated successfully, but these errors were encountered: