Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16105 )
Change subject: [docs] Add a section about using Gradle checkstyle ...................................................................... [docs] Add a section about using Gradle checkstyle Coming from Ant/Maven world, I was not aware of Gradle checkStyle plugin and didn't use it before submitting a patch to Gerrit. Knowing about checkStyle would have saved one Gerrit iteration. To save time for future first time Java committers, I added a section about using Gradle checkstyle task to check Java code style. Pre-rendered on my fork: https://github.com/grishick/kudu/blob/master/docs/contributing.adoc#java-code-style Change-Id: Ica45e5fce418fe40fbdcd354bb0c98dc0a653a65 Reviewed-on: http://gerrit.cloudera.org:8080/16105 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M docs/contributing.adoc 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16105 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ica45e5fce418fe40fbdcd354bb0c98dc0a653a65 Gerrit-Change-Number: 16105 Gerrit-PatchSet: 2 Gerrit-Owner: Greg Solovyev <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
