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 PackageName #4172

Closed
romani opened this Issue Apr 7, 2017 · 5 comments

Comments

Projects
None yet
4 participants
@romani
Member

romani commented Apr 7, 2017

subtask of #3891 .

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

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 7, 2017

Member

pre-assigned to @LubainaTariq .

@LubainaTariq , please make a comment "I am on it" to confirm assignment.

Member

romani commented Apr 7, 2017

pre-assigned to @LubainaTariq .

@LubainaTariq , please make a comment "I am on it" to confirm assignment.

@LubainaTariq

This comment has been minimized.

Show comment
Hide comment
@LubainaTariq

LubainaTariq commented Apr 8, 2017

I am on it

@LubainaTariq

This comment has been minimized.

Show comment
Hide comment
@LubainaTariq

LubainaTariq Apr 8, 2017

Please tell me in detail what I need to do and starting from which line of code?

LubainaTariq commented Apr 8, 2017

Please tell me in detail what I need to do and starting from which line of code?

@Vladlis

This comment has been minimized.

Show comment
Hide comment
@Vladlis

Vladlis Apr 8, 2017

Member

@LubainaTariq
There is a detailed description in the main issue: #3891
It also contains a lot of references from similar completed tasks for other checks.

Member

Vladlis commented Apr 8, 2017

@LubainaTariq
There is a detailed description in the main issue: #3891
It also contains a lot of references from similar completed tasks for other checks.

@rnveach

This comment has been minimized.

Show comment
Hide comment
@rnveach

rnveach May 2, 2017

Member

fix is merged

Member

rnveach commented May 2, 2017

fix is merged

@rnveach rnveach closed this May 2, 2017

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

timurt added a commit to timurt/checkstyle that referenced this issue May 6, 2017

timurt added a commit to timurt/checkstyle that referenced this issue May 6, 2017

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