-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
Add a --preview
flag to dry-run np
#391
Comments
Related: #249/#250 So it would kind of work, but only because (I think) nothing would be committed as the repo would be in a dirty state rather than (what I'd like) is to explicitly not push anything to GitHub |
One thing you could do at the moment to achieve this is to avoid setting an upstream branch, which would make |
Adding a dry run option does sound interesting and useful, though. We could add a |
--no-push
CLI flag to not push Git changes--preview
flag to dry-run np
@IssueHunt has funded $80.00 to this issue.
|
I just noticed the |
I don't understand 🤔. Did you mean: |
I believe not, because the NP does not show output of NPM. |
Maybe we could show parts of the output returned by |
@TiagoDanin Since, as you said above, the output of the command isn't really helpful and the option doesn't exist for Yarn, let's just print the publish command. |
Please also read through this issue about the It's super important to know for sure where you are publishing to when it might be private stuff. A simple misconfiguration somewhere might lead to publishing to the NPM registry which you might not want. |
@sindresorhus has rewarded $72.00 to @tiagodanin. See it on IssueHunt
|
Description
Is the feature request related to a problem?
I'd like to be able to run
np
in a "dry run" mode more so than what it currently does,np
already supports the--no-publish
flag to prevent it being published to npmjs.com, I'd also like to prevent any changes being pushed to GitHub, hence a--no-push
flag also.This would allow me to test
np
completely without pushing or publishing anything remotely to either npmjs.com or github.com.Possible solution
Follow up with a PR similar to that of ca0f193 where the
--no-publish
flag was added adding a new CLI option--no-push
Alternatives
IssueHunt Summary
tiagodanin has been rewarded.
Backers (Total: $80.00)
Submitted pull Requests
Tips
IssueHunt has been backed by the following sponsors. Become a sponsor
The text was updated successfully, but these errors were encountered: