Added small part with tips for IntelliJ settings to the Contrubuting.…

…md (#5148) * added small part with tips for IntelliJ settings * Update CONTRIBUTING.md Co-authored-by: Daniel Beck <[email protected]> * Apply suggestions from code review Co-authored-by: Oleg Nenashev <[email protected]> Co-authored-by: Daniel Beck <[email protected]> Co-authored-by: Oleg Nenashev <[email protected]>
jenkinsci · Jan 9, 2021 · ae6f5a9 · ae6f5a9
1 parent 3d297b3
commit ae6f5a9
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -115,6 +115,14 @@ the repository maintainers will integrate it, prepare changelogs, and
 ensure it gets released in one of upcoming Weekly releases.
 There is no additional action required from pull request authors at this point.
 
+## IntelliJ suggestion
+
+In case you are using IntelliJ, please adjust the default setting in respect to whitespace fixes on save.
+The setting can be found in Settings -> Editor -> General -> On Save -> Remove trailing spaces on: `Modified lines`
+This will help minimize the diff, which makes reviewing PRs easier.
+
+We also do not recommend `*` imports in the production code.
+Please disable them in Settings > Editor > Codestyle > Java by setting _Class count to use import with '*'_ and Names count to use import with '*'_ to a high value, e.g. 100. 
 ## Copyright
 
 The Jenkins core is licensed under [MIT license], with a few exceptions in bundled classes.