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

run_installer as an alias to run_always #711

Closed
MicaelJarniac opened this issue May 12, 2023 · 1 comment
Closed

run_installer as an alias to run_always #711

MicaelJarniac opened this issue May 12, 2023 · 1 comment

Comments

@MicaelJarniac
Copy link
Contributor

MicaelJarniac commented May 12, 2023

How would this feature be useful?

run_always has a misleading name.

Flag run run_always
No flag
-R
--install-only

It doesn't "always run", as the name implies.

Describe the solution you'd like

Add an alias to session.run_always, like session.run_installer, that makes it clear that it's meant to be used to run commands to install things. This would make sense with its current behavior of not running with -R.

This could then become the recommended method, and run_always could be kept for backward compatibility.

Describe alternatives you've considered

No response

Anything else?

No response

frostming pushed a commit to pdm-project/pdm that referenced this issue Jun 30, 2023
See wntrblm/nox#711

`session.run_always` works more similarly to `session.install`.
@cjolowicz
Copy link
Collaborator

Fixed in #770

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

No branches or pull requests

2 participants