Issue 49: Comments support in Checkstyle #189

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
3 participants
@baratali
Contributor

baratali commented Jul 14, 2014

Added support of single-line and block comments to Checkstyle.
Also edited TodoCommentCheck to show new functionality.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jul 14, 2014

Coverage Status

Coverage increased (+0.19%) when pulling f789408 on baratali:phase1 into 8c1b623 on checkstyle:master.

Coverage Status

Coverage increased (+0.19%) when pulling f789408 on baratali:phase1 into 8c1b623 on checkstyle:master.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jul 16, 2014

Member

@baratali , please squash last two commits in one.

Member

romani commented Jul 16, 2014

@baratali , please squash last two commits in one.

@romani romani self-assigned this Jul 16, 2014

@baratali

This comment has been minimized.

Show comment
Hide comment
@baratali

baratali Jul 16, 2014

Contributor

@romani , done.

Contributor

baratali commented Jul 16, 2014

@romani , done.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jul 16, 2014

Coverage Status

Coverage increased (+0.19%) when pulling f4cfaa3 on baratali:phase1 into d9452c8 on checkstyle:master.

Coverage Status

Coverage increased (+0.19%) when pulling f4cfaa3 on baratali:phase1 into d9452c8 on checkstyle:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jul 16, 2014

Coverage Status

Coverage increased (+0.19%) when pulling fe7c76b on baratali:phase1 into d9452c8 on checkstyle:master.

Coverage Status

Coverage increased (+0.19%) when pulling fe7c76b on baratali:phase1 into d9452c8 on checkstyle:master.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jul 18, 2014

Member

this PR is postponed till 5.9 release.

Member

romani commented Jul 18, 2014

this PR is postponed till 5.9 release.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 12, 2014

Coverage Status

Coverage increased (+0.19%) when pulling 4584f48 on baratali:phase1 into a0012d0 on checkstyle:master.

Coverage Status

Coverage increased (+0.19%) when pulling 4584f48 on baratali:phase1 into a0012d0 on checkstyle:master.

@baratali baratali changed the title from GSoC - Comments support in Checkstyle to Issue 49: Comments support in Checkstyle Aug 12, 2014

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Oct 17, 2014

Coverage Status

Coverage increased (+0.17%) when pulling 0d7814b on baratali:phase1 into a4ff027 on checkstyle:master.

Coverage Status

Coverage increased (+0.17%) when pulling 0d7814b on baratali:phase1 into a4ff027 on checkstyle:master.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Oct 18, 2014

Member

merged manually as fast-forward.

Member

romani commented Oct 18, 2014

merged manually as fast-forward.

@romani romani closed this Oct 18, 2014

@baratali baratali deleted the baratali:phase1 branch Oct 30, 2014

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