Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/11238 )
Change subject: [site] KUDU-1951 Exclude dirs from site_tool proof ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11238/2/site_tool File site_tool: http://gerrit.cloudera.org:8080/#/c/11238/2/site_tool@124 PS2, Line 124: '--file-ignore', '/apidocs|releases|configuration_ref|cpp-client-api|www/', > Maybe it would be better to use a separate checkout or multiple worktrees a I don't feel that strongly, I just think that at some point you'll end up with the entire listing of the files in master being duplicated into here or gitignore, no? eg why don't we have "src" or "java/" etc? -- To view, visit http://gerrit.cloudera.org:8080/11238 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: gh-pages Gerrit-MessageType: comment Gerrit-Change-Id: Iee421e4e218d40cef87db1e3f04991113afe596d Gerrit-Change-Number: 11238 Gerrit-PatchSet: 2 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Wed, 15 Aug 2018 22:08:57 +0000 Gerrit-HasComments: Yes
