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 MemberName #4151

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

Comments

Projects
None yet
3 participants
@romani
Member

romani commented Apr 3, 2017

subtask of #3891 .

do a copy of all test input files to separate folder for MemberName .
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 Gaurav Sharma.

Member

romani commented Apr 3, 2017

pre-assigned to Gaurav Sharma.

@gvsharma13

This comment has been minimized.

Show comment
Hide comment

I'm on it

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 4, 2017

Member

@gvsharma13 , please review in git history other commits, and do this task in similar way.
please read:
http://checkstyle.sourceforge.net/contributing.html
http://checkstyle.sourceforge.net/beginning_development.html

Member

romani commented Apr 4, 2017

@gvsharma13 , please review in git history other commits, and do this task in similar way.
please read:
http://checkstyle.sourceforge.net/contributing.html
http://checkstyle.sourceforge.net/beginning_development.html

@gvsharma13

This comment has been minimized.

Show comment
Hide comment
@gvsharma13

gvsharma13 Apr 16, 2017

Sir , is there time limit for this bug? Because it's my exam time. Can u extend ?

Sir , is there time limit for this bug? Because it's my exam time. Can u extend ?

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 18, 2017

Member

you have 1 month to fix it, in your case till 15th May.

Member

romani commented Apr 18, 2017

you have 1 month to fix it, in your case till 15th May.

@rnveach

This comment has been minimized.

Show comment
Hide comment
@rnveach

rnveach May 1, 2017

Member

Fix is merged

Member

rnveach commented May 1, 2017

Fix is merged

@rnveach rnveach closed this May 1, 2017

@rnveach rnveach added this to the 7.8 milestone May 1, 2017

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