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

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2020/07/23 08:16:20 UTC

[GitHub] [ignite] ivandasch commented on a change in pull request #8071: Make ducktape work with ISE / ISP

ivandasch commented on a change in pull request #8071:
URL: https://github.com/apache/ignite/pull/8071#discussion_r459284580



##########
File path: modules/ducktests/.gitignore
##########
@@ -0,0 +1,2 @@
+tests/venv/

Review comment:
       I believe that there is no need to  add .gitignore here at all.
   And I strongly advice you to use virtualenvwrapper in order do maintain different venv's.
   It is recommended way in python community to use venv's in developer's environment.
   If you used it it would not be necessary add something to .gitignore.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org