-
-
Notifications
You must be signed in to change notification settings - Fork 153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a command line option for setting python=False #301
Comments
I thought we already had this, but I'm not opposed to a venv backend that's a"passthrough". |
There seem to be something like this already : from the doc
However you cannot use I'll try to propose something to fix this |
PR #316 provides a solution to this: you can now use So @sethmlarson you could do what you wish using See #316 for details, there is also a new option to define the default venv backend. |
* New global option `nox.options.venv_backend` to set the default backend. Fixes #315 * Added doc about the new option * Blackened * fixed tests * Fixed docs * Fixed coverage by adding tests for the venv_backend_completer * fixed test * Added tests for short and long versions of the new option. * Replaced the venv_backend completer with a simple `choices` from argparse :) * Renamed venv_backend to default_venv_backend, and created new force_venv_backend * New "none" choice for venv_backends, equivalent to python=False * Updated doc concerning default_venv_backend and force_venv_backend, as well as the new 'none' backend * Fixed all manifest tests * Fixed test_tasks for venv_backend * Fixed coverage * Blackened code * The warning message was appearing for all sessions, even those deselected. It is now only logged when session is run. * Added `--no-venv` option. Fixes #301 * Blackened * Fixed tests * Improved coverage * Blackened * Fixed an issue with parametrization: warning would not be issued. Added corresponding tests. This should make coverage happy, too. * Blackened * Now `install` and `conda_install` work when there is no venv backend (or python=False). Previously a `ValueError` was raised. Fixes #318 * Fixed test * Minor edit to trigger CI build again as it seems stuck. * Minor doc fix to trigger the CI again (appveyor false fail) Co-authored-by: Sylvain MARIE <sylvain.marie@se.com>
How would this feature be useful?
If you have a
noxfile.py
that you use for development but also useDocker
as a part of your CI you can skip having to create a virtualenv and reinstall dependencies that may have been installed as a part of your Docker image build step. For instance if I have:then my
session.install("-r", "dev-requirements.txt")
step within mynoxfile.py
would only require checking cache, much faster to run.Describe the solution you'd like
Using
--use-current-env
(or alternative name, I'm fine with anything!) would override anypython=[]
withinnoxfile.py
topython=False
and use interpreter atsys.executable
.The text was updated successfully, but these errors were encountered: