Roman Ivanov comments

Results 1611 comments of


                                            Roman Ivanov

Issue #12011: tweet-releasenotes.sh should be able to work on start and finish release names

It is the same logic as at https://github.com/checkstyle/checkstyle/pull/12007#discussion_r945223740

SuppressWarningsHolder should allow usage of simple name of Checks in aliasList

Create unit test method to reproduce problem by Input file first. Second , update holder to have in map of names simple names also .

SuppressWarningsHolder should allow usage of simple name of Checks in aliasList

>Are you looking to match against CheckNameCheck or CheckName or both? This is good point we need both, user need to use same name of Check as he has in...

SuppressWarningsHolder should allow usage of simple name of Checks in aliasList

Please make test with values `name="aliasList" value="ParameterNumber=paramnum"` and will fail to suppress violation, so it is still not working.

SuppressWarningsHolder should allow usage of simple name of Checks in aliasList

Yes, this what is expected. Please send PR.

Feature request: Checks for local variable type inference

I see two problems: 1)Naming variable by names of methods used in expression 2) usage of var with name similar to type or "1)" To do something in checkstyle we...

Feature request: Checks for local variable type inference

Do you simply what to violate certain usage of var ? Like "no var for creating by new" ? And others.

Feature request: Checks for local variable type inference

@timurt , can your MatchXpath Check do this validation ?

Feature request: Checks for local variable type inference

@jensli , > disallow the use of var with certain types of initialiser expressions. is it what you expect ? ``` // If the method contains the name of the...

Feature request: Checks for local variable type inference

> forbidden "var" usage. you can do this by https://checkstyle.org/config_coding.html#IllegalType , as checkstyle treat `var` as special type.