New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split and Organize Checkstyle inputs by Test for LocalVariableName #4150

Closed
romani opened this Issue Apr 3, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@romani
Member

romani commented Apr 3, 2017

subtask of #3891 .

do a copy of all test input files to separate folder for LocalVariableName .
Not any other Check should use new inputs.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 3, 2017

Member

pre-assigned to @Shubhk15 .
@Shubhk15 , please make comment "I am on it" to confirm assignment.

Member

romani commented Apr 3, 2017

pre-assigned to @Shubhk15 .
@Shubhk15 , please make comment "I am on it" to confirm assignment.

@Kietzmann

This comment has been minimized.

Show comment
Hide comment
@Kietzmann

Kietzmann Apr 6, 2017

Contributor

I am on it

Contributor

Kietzmann commented Apr 6, 2017

I am on it

@Kietzmann

This comment has been minimized.

Show comment
Hide comment
@Kietzmann

Kietzmann Apr 10, 2017

Contributor

Please, make code review.
Thanks.

Contributor

Kietzmann commented Apr 10, 2017

Please, make code review.
Thanks.

romani added a commit that referenced this issue Apr 11, 2017

@romani romani added this to the 7.7 milestone Apr 11, 2017

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 11, 2017

Member

Fix is merged

Member

romani commented Apr 11, 2017

Fix is merged

@romani romani closed this Apr 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment