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

Do not skip pre-install commands when --install-only is passed #311

Merged
merged 2 commits into from
Mar 14, 2021

Conversation

cjolowicz
Copy link
Owner

@cjolowicz cjolowicz commented Mar 13, 2021

Use session.run_always to ensure the following commands are still run when --install-only is passed to Nox:

  • Exporting the lock file to constraints (poetry export)
  • Building the package (poetry build)
  • Reinstalling the package (pip uninstall)

nox-poetry now requires Nox 2020.8.22 or later, because the session.run_always method is not available in earlier versions.

Also, this PR fixes the Nox version constraint. The ^2020.5.24 constraint implies < 2021, preventing installation of nox-poetry with any Nox release newer than 2020.

@cjolowicz cjolowicz added the bug Something isn't working label Mar 13, 2021
@cjolowicz cjolowicz changed the title fix: Do not skip pre-install commands when --install-only is passed Do not skip pre-install commands when --install-only is passed Mar 13, 2021
@cjolowicz cjolowicz force-pushed the run-always branch 2 times, most recently from 97be416 to 7eb2799 Compare March 13, 2021 21:00
The `session.run_always` method is only available in 2020.8.22 and later.

Also, avoid the `^` version operator. It implies `< 2021`, preventing
installation of nox-poetry with any new Nox release.
@cjolowicz cjolowicz merged commit a68f918 into main Mar 14, 2021
@cjolowicz cjolowicz deleted the run-always branch March 14, 2021 01:07
This was referenced Mar 14, 2021
This was referenced Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant