Skip to content

Issue #11357: Report correct violation in lambda return#11409

Merged
romani merged 1 commit intocheckstyle:masterfrom
OttoKaaij:11357-Incorrect-violation-unnecessary-parentheses
Mar 17, 2022
Merged

Issue #11357: Report correct violation in lambda return#11409
romani merged 1 commit intocheckstyle:masterfrom
OttoKaaij:11357-Incorrect-violation-unnecessary-parentheses

Conversation

@OttoKaaij
Copy link
Contributor

@OttoKaaij OttoKaaij commented Mar 13, 2022

@OttoKaaij OttoKaaij force-pushed the 11357-Incorrect-violation-unnecessary-parentheses branch from 5666a6b to 87e7520 Compare March 13, 2022 11:57
Copy link
Contributor

@nrmancuso nrmancuso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please generate check regression report, you can find details here: https://github.com/checkstyle/contribution/tree/master/checkstyle-tester#report-generation

@nrmancuso nrmancuso self-assigned this Mar 13, 2022
@OttoKaaij OttoKaaij force-pushed the 11357-Incorrect-violation-unnecessary-parentheses branch from 87e7520 to 11c1982 Compare March 13, 2022 12:38
@OttoKaaij OttoKaaij force-pushed the 11357-Incorrect-violation-unnecessary-parentheses branch 2 times, most recently from c8b6bf5 to cdec517 Compare March 13, 2022 12:46
@OttoKaaij
Copy link
Contributor Author

@nick-mancuso Before I generate the regression report, could you check if the config is correct like this?

@nrmancuso
Copy link
Contributor

@nick-mancuso Before I generate the regression report, could you check if the config is correct like this?

Config looks good, you don't have to specify the projects file, we have default for github action.

@OttoKaaij
Copy link
Contributor Author

GitHub, generate report

@github-actions
Copy link
Contributor

@OttoKaaij OttoKaaij force-pushed the 11357-Incorrect-violation-unnecessary-parentheses branch from cdec517 to fc17e52 Compare March 13, 2022 14:53
@OttoKaaij
Copy link
Contributor Author

Please extend your new input file to include cases like:

Should be done!

@nrmancuso
Copy link
Contributor

Ci failure is not related, but is consistent: https://checkstyle.semaphoreci.com/jobs/92d3c7fd-7382-4984-991a-4ee4eb4da260

Execution default of goal org.antlr:antlr4-maven-plugin:4.9.3:antlr4 failed: A required class was missing while executing org.antlr:antlr4-maven-plugin:4.9.3:antlr4: org/stringtemplate/v4/STGroup

I suppose this is a problem downloading some ANTLR dependency.

@nrmancuso nrmancuso requested a review from strkkk March 13, 2022 15:17
@nrmancuso nrmancuso assigned strkkk and unassigned nrmancuso Mar 13, 2022
@strkkk strkkk requested a review from pbludov March 15, 2022 02:28
@strkkk strkkk assigned pbludov and unassigned strkkk Mar 15, 2022
@pbludov pbludov assigned romani and unassigned pbludov Mar 16, 2022
@romani romani merged commit c2ed449 into checkstyle:master Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UnnecessaryParentheses reports incorrect violation message for return value in lambdas

5 participants