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

List all sections when a list of sections is set using nox.options #161

Closed
pradyunsg opened this issue Dec 12, 2018 · 3 comments · Fixed by #185
Closed

List all sections when a list of sections is set using nox.options #161

pradyunsg opened this issue Dec 12, 2018 · 3 comments · Fixed by #185

Comments

@pradyunsg
Copy link
Contributor

Describe the bug
Setting nox.options.sessions results in nox -l only those sections, not all the sections.

How to reproduce
Create a noxfile.py with 3 sessions (format, tests, docs) and set nox.options.sessions = ["format"], then run nox -l:

Available sessions:
* format

Expected behavior
It would be better if nox -l would actually list all the sessions.


Related Feature Request: The sessions that are run by default be marked with a * while the others won't have a bullet:

Available sessions:
* format
  tests
  docs
@theacodes
Copy link
Collaborator

Good catch. Filtering happens before listing. I'd like to keep that behavior for the -k argument but not for the -s argument.

@pradyunsg
Copy link
Contributor Author

The -s docstring also needs to be updated for the change in behavior possible via the options attribute?

It currently says that all sessions are run when -s is not given, but currently that is not the case anymore.

@theacodes
Copy link
Collaborator

theacodes commented Dec 14, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants