Attila Bukor 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/', > hm, I guess it's just me, but I always have 30+ untracked non-ignored files I have several non-tracked/non-ignored files too, but not all of them break stuff (this one's breaking proof because it contains a trace.html). Also, I only wanted to ignore files/directories that seem to be common enough to be annoying for multiple people. Anyway, you're right that we should draw the line before we end up ignoring '*', so I'll remove this one -- 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:18:22 +0000 Gerrit-HasComments: Yes
