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

Support frozen install of only production dependencies #254

Closed
3 tasks done
BorePlusPlus opened this issue Jan 27, 2025 · 0 comments · Fixed by #255
Closed
3 tasks done

Support frozen install of only production dependencies #254

BorePlusPlus opened this issue Jan 27, 2025 · 0 comments · Fixed by #255
Labels
enhancement New feature or request

Comments

@BorePlusPlus
Copy link
Contributor

BorePlusPlus commented Jan 27, 2025

Clear and concise description of the problem

As mentioned in the additional context section of #250, it is useful to be able to install only production dependencies (by passing -P flag) while adhering to the frozen install semantics.
This allows for building releases with just production dependencies while ensuring consistent versions of dependencies.

Suggested solution

Allow passing -P parameter when calling nci or ni --frozen.
I will prepare a PR for it.

Alternative

I'm not quite sure - possibly a separate command (e.g. ncip) that encapsulates it all? The benefit would be tighter control over input arguments. The proposed approach allows for the passing of any additional arguments to nci or ni --frozen, not just the -P.

Additional context

Requires a syncing release of antfu-collective/package-manager-detector#40, which enables arguments to be passed to the frozen command in the first place.

I possibly find it a bit too amusing that the combination of flags --frozen and -P in ni --frozen -P reads as frozen pea 🤭

Validations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant