Skip to content

Add a --preview flag to dry-run np #391

Closed

Description

@ntwb

Issuehunt badges

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 tiagodanin has been rewarded.

Backers (Total: $80.00)

Submitted pull Requests


Tips


IssueHunt has been backed by the following sponsors. Become a sponsor

Activity

  1. ntwb commented on Apr 18, 2019

    @ntwb
    Author

    Related: #249/#250 -no-git-tag-version is similar/close to what I'm asking above, though the PR #250 would still push to GitHub which is what I'd like to avoid.

    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

  2. itaisteinherz commented on Apr 18, 2019

    @itaisteinherz
    Collaborator

    One thing you could do at the moment to achieve this is to avoid setting an upstream branch, which would make np skip pushing to the remote (however this is obviously a hack, not a solution).
    Note that this is also related to #386.

  3. itaisteinherz commented on Apr 18, 2019

    @itaisteinherz
    Collaborator

    Adding a dry run option does sound interesting and useful, though. We could add a --preview flag which only prints out the changes/actions np is going to make without actually doing those.

  4. changed the title [-]Feature Request: Add a `--no-push` CLI flag to not push Git changes[/-] [+]Add a `--preview` flag to dry-run np[/+] on Apr 18, 2019
  5. IssueHuntBot commented on May 2, 2019

    @IssueHuntBot

    @IssueHunt has funded $80.00 to this issue.


  6. itaisteinherz commented on May 8, 2019

    @itaisteinherz
    Collaborator

    I just noticed the --dry-run flag for npm publish, which could be useful here.

  7. TiagoDanin commented on May 10, 2019

    @TiagoDanin
    Contributor

    I just noticed the --dry-run flag for npm publish, which could be useful here.

    I don't understand 🤔. Did you mean: --no-publish ?

  8. itaisteinherz commented on May 10, 2019

    @itaisteinherz
    Collaborator

    No, this is what I meant:

    Screenshot

  9. TiagoDanin commented on May 10, 2019

    @TiagoDanin
    Contributor

    I believe not, because the NP does not show output of NPM.

  10. itaisteinherz commented on Jun 29, 2019

    @itaisteinherz
    Collaborator

    I believe not, because the NP does not show output of NPM.

    Maybe we could show parts of the output returned by npm publish --dry-run, or even the whole output (I'm not sure if listr will allow us to do this though).

  11. TiagoDanin commented on Jul 8, 2019

    @TiagoDanin
    Contributor

    DeepinScreenshot_select-area_20190702111444

    Name and version are already displayed in NP. And the output of --dry-run is immediately displayed (no waiting), listr only show one line at a time.

    Another problem is that Yarn has no option --dry-run.

    PS: Sorry for the delay.

  12. itaisteinherz commented on Jul 9, 2019

    @itaisteinherz
    Collaborator

    @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.

  13. cactyx commented on Jul 15, 2019

    @cactyx

    Please also read through this issue about the npm publish command.

    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.

  14. issuehunt-oss commented on Feb 22, 2020

    @issuehunt-oss

    @sindresorhus has rewarded $72.00 to @tiagodanin. See it on IssueHunt

    • 💰 Total deposit: $80.00
    • 🎉 Repository reward(0%): $0.00
    • 🔧 Service fee(10%): $8.00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions