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

Update Checkstyle GUI in order to display Javadoc tokens #408

Closed
baratali opened this Issue Dec 2, 2014 · 6 comments

Comments

Projects
None yet
3 participants
@baratali
Contributor

baratali commented Dec 2, 2014

Need to parse block comments as Javadoc comments and show Javadoc DetailNode tree instead of block comment.
screenshot

@rnveach

This comment has been minimized.

Show comment
Hide comment
@rnveach

rnveach Feb 14, 2016

Member

@baratali Are you still working on this?

Member

rnveach commented Feb 14, 2016

@baratali Are you still working on this?

@baratali

This comment has been minimized.

Show comment
Hide comment
@baratali

baratali Feb 14, 2016

Contributor

@rnveach I'm working on #652 now

Contributor

baratali commented Feb 14, 2016

@rnveach I'm working on #652 now

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Feb 16, 2016

Member

as soon we put Javadoc nodes to parse tree , GUI will show them automatically.

Member

romani commented Feb 16, 2016

as soon we put Javadoc nodes to parse tree , GUI will show them automatically.

@baratali

This comment has been minimized.

Show comment
Hide comment
@baratali

baratali Aug 14, 2016

Contributor

Before providing a pull-request I'd like to disscuss GUI layout.
I added a new dropdown next to "Reload" button. User can select parsing mode (whether a file should be parsed without comments/with comments/with Javadocs).

@romani , is it OK?

Contributor

baratali commented Aug 14, 2016

Before providing a pull-request I'd like to disscuss GUI layout.
I added a new dropdown next to "Reload" button. User can select parsing mode (whether a file should be parsed without comments/with comments/with Javadocs).

@romani , is it OK?

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Aug 15, 2016

Member

Looks good

Member

romani commented Aug 15, 2016

Looks good

baratali added a commit to baratali/checkstyle that referenced this issue Sep 5, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 6, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 6, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 6, 2016

@romani romani added the approved label Sep 6, 2016

@romani romani added this to the 7.2 milestone Sep 6, 2016

@romani romani added the miscellaneous label Sep 6, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 6, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 6, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 8, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 9, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 10, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 10, 2016

baratali added a commit to baratali/checkstyle that referenced this issue Sep 10, 2016

romani added a commit that referenced this issue Sep 11, 2016

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Sep 11, 2016

Member

fix is merged.

Member

romani commented Sep 11, 2016

fix is merged.

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