Quantcast

Is there enum naming check?

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Is there enum naming check?

Pavel Baranchikov
Hello,

        Does checkstyle have enum elements naming check, similar to MemberName or
LocalFinalVariableName?

Best regards,
Pavel Baranchikov
Thursday 23 January 2014

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Checkstyle-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Is there enum naming check?

Roman Ivanov
Hi Pavel,

I cannot find any reference to ENUM_DEF or  ENUM_CONSTANT_DEF at
http://checkstyle.sourceforge.net/config_naming.html
and sources https://github.com/checkstyle/checkstyle/tree/master/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/naming

Switched all naming checks and filed to make force validate Enum names of its constants.

looks like good point to improve checkstyle:
updates for ConstantName module for  ENUM_CONSTANT_DEF and  TypeName module for ENUM_DEF .

FYI, new location of mail-list is there (sourceforce mail-list will be closed after 5.7 release, that is very soon):
https://groups.google.com/forum/#!forum/checkstyle
https://groups.google.com/forum/#!forum/checkstyle-devel
https://groups.google.com/forum/#!forum/checkstyle-announce

thanks,
Roman Ivanov
------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Checkstyle-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Is there enum naming check?

Pavel Baranchikov
Hello,

        Ok, let me do the implementation.

> Hi Pavel,
>
> I cannot find any reference to ENUM_DEF or  ENUM_CONSTANT_DEF at
> http://checkstyle.sourceforge.net/config_naming.html
> and sources
> https://github.com/checkstyle/checkstyle/tree/master/src/checkstyle/com/pup
> pycrawl/tools/checkstyle/checks/naming
>
> Switched all naming checks and filed to make force validate Enum names of
> its constants.
>
> looks like good point to improve checkstyle:
> updates for ConstantName module for  ENUM_CONSTANT_DEF and  TypeName module
> for ENUM_DEF .
>
> FYI, new location of mail-list is there (sourceforce mail-list will be
> closed after 5.7 release, that is very soon):
> https://groups.google.com/forum/#!forum/checkstyle
> https://groups.google.com/forum/#!forum/checkstyle-devel
> https://groups.google.com/forum/#!forum/checkstyle-announce
>
> thanks,
> Roman Ivanov
> ----------------------------------------------------------------------------
> -- CenturyLink Cloud: The Leader in Enterprise Cloud Services.
> Learn Why More Businesses Are Choosing CenturyLink Cloud For
> Critical Workloads, Development Environments & Everything In Between.
> Get a Quote or Start a Free Trial Today.
> http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
> _______________________________________________
> Checkstyle-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/checkstyle-devel

 

Best regards,
Pavel Baranchikov
Friday 24 January 2014

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Checkstyle-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Is there enum naming check?

Roman Ivanov
Hi Pavel,

remind you one more time to avoid misunderstanding after new functionality is completed.

https://groups.google.com/forum/#!topic/checkstyle-devel/JbhfiiSqKsQ

If that changes end up in new Check or complicated update - it have to be done in sanbox/experimental project to ease it testing.

All discussions should be continued in google mail-list.

Thanks,
Roman Ivanov

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Checkstyle-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel
Loading...