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

AnnotationLocationcheck parameter "setAllowSamelineParametrizedAnnotation" is a typo #585

Closed
hgschmie opened this Issue Jan 18, 2015 · 5 comments

Comments

Projects
None yet
4 participants
@hgschmie

hgschmie commented Jan 18, 2015

That should be setAllowSamelineParameterizedAnnotation (missing 'e').

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 18, 2015

Member

all sources(javadoc), xdocs, configs should be changed.

Member

romani commented Jan 18, 2015

all sources(javadoc), xdocs, configs should be changed.

alexkravin added a commit to alexkravin/checkstyle that referenced this issue Jan 20, 2015

alexkravin added a commit to alexkravin/checkstyle that referenced this issue Jan 20, 2015

alexkravin added a commit to alexkravin/checkstyle that referenced this issue Jan 21, 2015

alexkravin added a commit to alexkravin/checkstyle that referenced this issue Jan 21, 2015

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 21, 2015

Member

will be in 6.3

Member

romani commented Jan 21, 2015

will be in 6.3

@romani romani closed this Jan 21, 2015

@lkoe

This comment has been minimized.

Show comment
Hide comment
@lkoe

lkoe Feb 20, 2015

Member

The fix is not quite complete, the member variable is changed alright - but the corresponding setter is not.
This means configurations will still need to use the old, wrong property name - contradicting the current docs for Checkstyle 6.3.

Member

lkoe commented Feb 20, 2015

The fix is not quite complete, the member variable is changed alright - but the corresponding setter is not.
This means configurations will still need to use the old, wrong property name - contradicting the current docs for Checkstyle 6.3.

@alexkravin

This comment has been minimized.

Show comment
Hide comment
@alexkravin

alexkravin Feb 21, 2015

Contributor

completed the fix.

Contributor

alexkravin commented Feb 21, 2015

completed the fix.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Feb 21, 2015

Member

will be in 6.4

Member

romani commented Feb 21, 2015

will be in 6.4

@romani romani closed this Feb 21, 2015

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