search

l3r8yJ / oop-cop

The static analysis tool and a Maven plugin that will help you model your objects, classes, methods properly by rejecting your non-perfect code.
https://www.l3r8y.ru/oop-cop/
MIT License
9 stars 5 forks source link

rename `maxClassNameLen` to `LongClassNameCheck` #111

Closed h1alexbel closed 8 months ago

h1alexbel commented 8 months ago

how about renaming maxClassNameLen maven plugin parameter to LongClassNameCheck?

I have idea of printing the name of the check that was failed:

blah blah blah... (LongClassNameCheck)

@l3r8yJ what do you think?

zoeself commented 8 months ago

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

zoeself commented 8 months ago

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-01-13T07:10:04.380653.

Estimation here is 60 minutes, that's how much you will be paid.

Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.

If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.

If you don't have time or simply don't want to work on this, you can always resign.

l3r8yJ commented 8 months ago

@h1alexbel LongClassNameCheck contains check and i'm not sure that we need to assign some value to check

h1alexbel commented 8 months ago

@l3r8yJ yeah, that's a reasonable point

zoeself commented 8 months ago

@l3r8yJ thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.