Skip to content

Commit

Permalink
[maintenance] auto-formatting via CLI hint (#301)
Browse files Browse the repository at this point in the history
  • Loading branch information
Tomasz Pasternak committed Sep 29, 2022
1 parent 3713825 commit 2026c61
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions docs/dev/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@
_You can download a formatter for Intellij [here](https://plugins.jetbrains.com/plugin/8527-google-java-format)._ \
**NOTE: there might be a problem with imports order - static imports have to before other imports. You can change it
in the IntelliJ settings.**

- Alternatively, you can run:
```
cs launch com.google.googlejavaformat:google-java-format:<version> -- -r @<(git diff --name-only master| grep "[.]java")
```
- `io.vavr` provides great collection / control types - use them!
- `null` is bad, if something may be nullable / may not exist use `io.vavr.control.Option`.
- use `io.vavr.collection.List` instead of `java.util.List` - it allows using `.map` without `.stream()`...
Expand All @@ -19,4 +22,4 @@
- remember to use [buildifier](https://github.com/bazelbuild/buildtools/blob/master/buildifier/README.md). \
_The usage: `buildifier -r .`_
- remember to update the [**changelog**](../../CHANGELOG.md).
- **tests tests tests** - write tests!
- **tests tests tests** - write tests!

0 comments on commit 2026c61

Please sign in to comment.