Skip to content
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.

Sign up for GitHub

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

Jump to bottom

move all methods from checkstyle.api.Utils to checkstyle.Utils #661

Closed
romani opened this issue Feb 21, 2015 · 1 comment
Closed

move all methods from checkstyle.api.Utils to checkstyle.Utils #661

romani opened this issue Feb 21, 2015 · 1 comment
Assignees
@alexkravin
Labels
approved

Comments

@romani
Copy link
Member

romani commented Feb 21, 2015

#660 should be resolved first.

Util methods implementations are not api, it is not contract. Signature of methods could be contract/api but not a reallization.

Please move all methods as is ( without changes) out of api package to com/puppycrawl/tools/checkstyle/Utils.java.

It is required to analyse sources of all know plugins on the usage api.Utils class and provide report on that, all affcted prpojects should be notified.
alexkravin added a commit to alexkravin/checkstyle that referenced this issue Mar 13, 2015
@alexkravin
Moved all methods from checkstyle.api.utils to checkstyle.utils, issue …
1e836d8 
…checkstyle#661
@alexkravin alexkravin mentioned this issue Mar 13, 2015
Closed
alexkravin added a commit to alexkravin/checkstyle that referenced this issue Mar 13, 2015
@alexkravin
Moved all methods from checkstyle.api.utils to checkstyle.utils, issue …
ab21cec 
…checkstyle#661
alexkravin added a commit to alexkravin/checkstyle that referenced this issue Mar 15, 2015
@alexkravin
Moved all methods from checkstyle.api.utils to checkstyle.utils, issue …
f436eb7 
…checkstyle#661
@alexkravin alexkravin mentioned this issue Mar 15, 2015
Closed
alexkravin added a commit to alexkravin/checkstyle that referenced this issue Mar 15, 2015
@alexkravin
Moved all methods from checkstyle.api.utils to checkstyle.utils, issue …
f643278 
…checkstyle#661
romani pushed a commit that referenced this issue Mar 15, 2015
@alexkravin @romani
Moved all methods from checkstyle.api.utils to checkstyle.utils, issue …
1c15b6a 
…#661
@romani
Copy link
Member Author

romani commented Mar 15, 2015

fixed, will be in 6.5

@romani romani closed this as completed Mar 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexkravin alexkravin

Labels
approved
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@romani @alexkravin