Adar Dembo has posted comments on this change.

Change subject: [python] - Expand gitignore for python
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/4616/1/.gitignore
File .gitignore:

Since all of this stuff is rooted in python/, can you make a second .gitignore 
file in the python subdirectory and put it all in there? Actually, one already 
exists; can you just add to it?


Line 74: *.egg
How could egg files leak out of the python subdirectory? That seems like 
something we should fix.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4b7bcac00a88678a555a4d85419c22decac23b58
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to