Let's print actual class name length

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

Let's print actual class name length #175

Closed h1alexbel closed 3 months ago

h1alexbel commented 3 months ago

I got this error:

'DeepInfraRequest' has bad naming, class name is more than 15, consider more simple name
[ERROR]     read: 'https://www.yegor256.com/2015/01/12/compound-name-is-code-smell.html' (LongClassNameCheck)

This error only states that 'DeepInfraRequest' class name length is more than 15 chars, but does not print what is the actual length of this class. It can improve developer experience with oop-cop error handling

zoeself commented 3 months ago

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

h1alexbel commented 3 months ago

@l3r8yJ WDYT?

l3r8yJ commented 3 months ago

@h1alexbel thanks for the report, but here some unclear moments:

It's not clear what is the actual length of the class name (Bug report does not provide the actual length). Consider adding the actual length of the class name to the error message.

Generated with Phind/Phind-CodeLlama-34B-v2

l3r8yJ commented 3 months ago

@h1alexbel good suggestion

zoeself commented 3 months ago

@l3r8yJ I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.

Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

l3r8yJ commented 3 months ago

@rultor release, tag is 0.2.9

rultor commented 3 months ago

@rultor release, tag is 0.2.9

@l3r8yj OK, I will release it now. Please check the progress here

rultor commented 3 months ago

@rultor release, tag is 0.2.9

@l3r8yj Done! FYI, the full log is here (took me 7min)