You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2017/09/01 00:00:06 UTC

[kudu-CR] [doc] add info about the iwyu target

Alexey Serbin has posted comments on this change.

Change subject: [doc] add info about the iwyu target
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/7928/2/README.adoc
File README.adoc:

PS2, Line 264:  
> missing word: "the"
Done


PS2, Line 267: w
> nit: capitalize Why because it's the title of an article
Done


-- 
To view, visit http://gerrit.cloudera.org:8080/7928
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I87da493486500cde0cd226614f8a19985d295a96
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-HasComments: Yes