Skip to content

Commit f2660be

Browse files
authored
Merge pull request #253 from psycho-ir/code-styles
Code styles
2 parents cfbbac3 + 39f7783 commit f2660be

File tree

32 files changed

+1101
-74
lines changed

32 files changed

+1101
-74
lines changed

CONTRIBUTING.md

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,22 @@ All PRs have to be reviewed and signed off by another developer before being mer
4848
branch. This review will likely ask for some changes to the code - please don't be alarmed or upset
4949
at this; it is expected that all PRs will need tweaks and a normal part of the process.
5050

51+
The PRs are checked to be compliant with the Java Google code style.
52+
5153
Be aware that all Operator SDK code is released under the [Apache 2.0 licence](LICENSE).
5254

53-
### Thanks
55+
## Development environment setup
56+
57+
### Code style
58+
59+
The SDK modules and samples are formatted to follow the Java Google code style.
60+
On every `compile` the code gets formatted automatically,
61+
however, to make things simpler (i.e. avoid getting a PR rejected simply because of code style issues), you can import one of the following code style schemes based on the IDE you use:
62+
63+
- for *Intellij IDEA* import [contributing/intellij-google-style.xml](contributing/intellij-google-style.xml)
64+
- for *Eclipse* import [contributing/eclipse-google-style.xml](contributing/eclipse-google-style.xml)
65+
66+
## Thanks
5467

5568
These guidelines were best on several sources, including
5669
[Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md), [PurpleBooth's

contributing/eclipse-google-style.xml

Lines changed: 337 additions & 0 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)