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

Add a formatter #975

Closed
wants to merge 6 commits into from
Closed

Add a formatter #975

wants to merge 6 commits into from

Conversation

@AnEmortalKid
Copy link
Contributor

@AnEmortalKid AnEmortalKid commented Apr 20, 2020

I configured the https://github.com/revelc/formatter-maven-plugin with the sun style conventions. The only two changes I made to those formatter settings were to set "spaces" as the only indentation and to set the indent level to 4.

The change of formatting all the sources is in this commit: fbdd711

Hopefully this makes it easier for contributors to not have to worry about sun style!

I might have to adjust the line length it seems since a bunch of the javadoc got changed. If you have the number you use, I'll set it to that.

Jan Monterrubio and others added 6 commits Apr 18, 2020
@AnEmortalKid
Copy link
Contributor Author

@AnEmortalKid AnEmortalKid commented May 22, 2020

@wumpz are you interested in a formatter for the project?

@AnEmortalKid AnEmortalKid deleted the AnEmortalKid:formatter branch Jun 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.