Split and Organize Checkstyle inputs by Test for WhitespaceAround #3898

Closed
romani opened this Issue Mar 2, 2017 · 6 comments

Comments

Projects
None yet
4 participants
@romani
Member

romani commented Mar 2, 2017

subtask of #3891 .

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

@romani romani added the approved label Mar 2, 2017

@romani

This comment has been minimized.

Show comment
Hide comment
@nikshinde1996

This comment has been minimized.

Show comment
Hide comment
@nikshinde1996

nikshinde1996 Mar 2, 2017

I am on it !

I am on it !

@nikshinde1996

This comment has been minimized.

Show comment
Hide comment
@nikshinde1996

nikshinde1996 Mar 11, 2017

I have followed all the steps provided by your description.. Then how come build failed. Please guide me to fix this issue.

I have followed all the steps provided by your description.. Then how come build failed. Please guide me to fix this issue.

@rnveach

This comment has been minimized.

Show comment
Hide comment
Member

rnveach commented Mar 11, 2017

@nikshinde1996 See my comments at #3982 (comment)

@shivanshsoni

This comment has been minimized.

Show comment
Hide comment
@shivanshsoni

shivanshsoni Mar 16, 2017

Contributor

@nikshinde1996 Are you still working on this issue?

Contributor

shivanshsoni commented Mar 16, 2017

@nikshinde1996 Are you still working on this issue?

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 11, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 12, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 12, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 12, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 12, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 13, 2017

Kietzmann added a commit to Kietzmann/checkstyle that referenced this issue Apr 13, 2017

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

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

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Apr 13, 2017

Member

fix is merged

Member

romani commented Apr 13, 2017

fix is merged

@romani romani closed this Apr 13, 2017

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