Skip to content

Apply checkstyle to force coding style#272

Open
eincs wants to merge 5 commits into
objectify:masterfrom
eincs:apply-checkstyle
Open

Apply checkstyle to force coding style#272
eincs wants to merge 5 commits into
objectify:masterfrom
eincs:apply-checkstyle

Conversation

@eincs

@eincs eincs commented Oct 9, 2015

Copy link
Copy Markdown
Contributor

Checkstyle is well-known and popular coding style forcing tool.
Build will failed if srouce code violoates condition that defines on checkstyle.xml .
Many other open-source based on java uses Checkstyle, and Checkstyle will help to maintain code quality of the project.

I add some condition for Objectify, and other condtions can be applied step-by-step.
By merging pull-reqeust, future violation of these condition will be cause build failure:

  • All the source code must end with newline character.
  • Trailing whitespace must not exsits.
  • Does not allow two or more consecutive empty lines.
  • Deos not allow an empty line before '}'
  • Does not allow an empty line after a short line that ends with '{'"

@eincs eincs changed the title Apply checkstyle for force coding style Apply checkstyle to force coding style Oct 9, 2015
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.

1 participant