search

tox4j / deprecated-tox4j

Deprecated, see http://github.com/TokTok/jvm-toxcore-c
http://github.com/TokTok/jvm-toxcore-c
Other
42 stars 17 forks source link

Indent #72

Closed iphydf closed 9 years ago

iphydf commented 9 years ago

This is intended to be a place for style discussion and frequent rebases until we reach consensus.

ghost commented 9 years ago

Welcome, @iphydf, thanks for contributing to tox4j! @sonOfRa has been assigned to this review, and will get to you shortly about any necessary changes to your Pull Request. In the meantime, our CI will build your Pull Request and report any errors.

sonOfRa commented 9 years ago

@iphydf Can you make the ToxOptions change a separate Pull Request later on? Let's keep this Pull Request strictly to checkstyle rules and formatting.

iphydf commented 9 years ago

Yes, I did. However, the ToxOptions change is result of checkstyle recommendations, so this PR is only indent.

sonOfRa commented 9 years ago

@toxbot r+ ecc58f4

ghost commented 9 years ago

:hourglass: Testing commit ecc58f4 with merge 8b97195...

ghost commented 9 years ago

:sunny: Test successful - travis