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.

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

fixed checkstyle/eclipse throws indent impedance #1

Closed
wants to merge 1 commit into from
Closed

fixed checkstyle/eclipse throws indent impedance #1

wants to merge 1 commit into from

Conversation

ksclarke
Copy link
Contributor

https://wiki.duraspace.org/display/FF/Code+Style+Guide?focusedCommentId=34656139#comment-34656139 notes that there is an impedance mismatch between the Eclipse formatter rules and Checkstyle.

This has been fixed in Checkstyle now (checkstyle/checkstyle#80) and a new 5.7 release is available with the ability to set a throwsIndent value. The eclipse-cs plugin has also been updated to incorporate the new Checkstyle release.

This PR adds the throwsIndent setting to the fcrepo checkstyle rules so that they will match the standard Eclipse and Java formatting standard.

@ksclarke
Copy link
Contributor Author

ksclarke commented May 2, 2014

Closing because we decided to wait until the other checkstyle-maven-plugin bug was resolved before taking action.

@ksclarke ksclarke closed this May 2, 2014
@ksclarke ksclarke deleted the throwsIndent branch May 2, 2014 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant