Skip to content
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

Update Noxfiles to use new Nox #6170

Closed
theacodes opened this issue Oct 5, 2018 · 6 comments
Closed

Update Noxfiles to use new Nox #6170

theacodes opened this issue Oct 5, 2018 · 6 comments
Assignees
Labels
testing type: cleanup An internal cleanup or hygiene concern.

Comments

@theacodes
Copy link
Contributor

theacodes commented Oct 5, 2018

Nox recently made a big change from declarative to imperative and broke the Noxfile API.

We need to update our Noxfiles to work with the new API and update our test scripts to install nox (instead of nox-automation) and invoke python3 -m nox {args}.

Most projects just need to do a few steps:

  1. Remove usage of session_{name} sessions and use @nox.session instead.
  2. Don't use session.interpreter, use @nox.session(py='...')
  3. Don't use nox.parametrize for multiple Pythons, use @nox.session(py=['..', '..']).
@theacodes theacodes added testing type: cleanup An internal cleanup or hygiene concern. labels Oct 5, 2018
@busunkim96 busunkim96 mentioned this issue Oct 8, 2018
@tseaver
Copy link
Contributor

tseaver commented Oct 11, 2018

Reopening: the root noxfile needs updating as well.

@tseaver tseaver reopened this Oct 11, 2018
@tseaver
Copy link
Contributor

tseaver commented Oct 11, 2018

@theacodes it looks like making the new nox work on CircleCI (e.g., to build docs) needs a new Docker image, as well.

@theacodes
Copy link
Contributor Author

I'll leave that up to @busunkim96. Holler if y'all need help.

@tseaver
Copy link
Contributor

tseaver commented Oct 11, 2018

@busunkim96 #6196 has a patch for the nox.py -> noxfile.py part, but don't have the keys / know-how to publish a new Docker image to googleapis/nox.

@theacodes
Copy link
Contributor Author

theacodes commented Oct 11, 2018 via email

@tseaver
Copy link
Contributor

tseaver commented Oct 11, 2018

AFAICT, the .kororo config doesn't specify a Docker image: instead, it forcibly updates nox:

# Remove old nox
python3.6 -m pip uninstall --yes --quiet nox-automation

# Install nox
python3.6 -m pip install --upgrade --quiet nox
python3.6 -m nox --version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Development

No branches or pull requests

3 participants