-
Notifications
You must be signed in to change notification settings - Fork 0
Make use of tox-conda optional #11
Comments
@saimn this is a good idea. Can you share an example of the error that occurs? Do you think it would make sense to have another config option like |
We could also imagine providing a command-line option to enable/disable |
The error is if I install tox-conda on a non-conda env (I forgot to mention this above, sorry):
It's not all clear for me how to best manage this, and having a tox conf working on multiple environments. |
Ah, okay. That's a case I hadn't considered. So you'd like it to automatically determine if
This should be true. Lines 60 to 65 in 8cb9d2f
We could allow it to do this automatically when using a |
About the first issue, when using tox-conda in a non-conda setup, I'm not sure what is the best. Thinking more about, just raising an error saying that conda is not found is probably the best solution, so what is done now. |
Currently if tox-conda is installed and conda-deps is defined, this raises an error.
To make it easier to share a tox config for conda and non-conda envs, it would be useful to either continue with the default tox process, or maybe better to have a setting to activate tox-conda on a per env basis ?
The text was updated successfully, but these errors were encountered: