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

fix: update notification includes -g flag for cli #150

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

olizilla
Copy link
Contributor

The update notifier now specifies the global flag in the npm install advice as this is a cli.

   ╭────────────────────────────────────────────────╮
   │                                                │
   │        Update available 7.0.0 → 7.0.3          │
   │   Run npm i -g @web3-storage/w3cli to update   │
   │                                                │
   ╰────────────────────────────────────────────────╯

before this change, it would encourage you to do a local install which is probably not what you want

License: MIT

The helpful update notifier now specifies the global flag in the npm install advice as this is a cli.

before this change, it would encourage you to do a local install which is probably not what you want

```console
   ╭─────────────────────────────────────────────╮
   │                                             │
   │       Update available 7.0.0 → 7.0.3        │
   │   Run npm i @web3-storage/w3cli to update   │
   │                                             │
   ╰─────────────────────────────────────────────╯
```

License: MIT
Signed-off-by: Oli Evans <[email protected]>
@olizilla olizilla merged commit 370bfc6 into main Dec 15, 2023
1 check passed
@olizilla olizilla deleted the update-notify-global branch December 15, 2023 17:26
alanshaw pushed a commit that referenced this pull request Jan 9, 2024
🤖 I have created a release *beep* *boop*
---


##
[7.0.4](v7.0.3...v7.0.4)
(2024-01-09)


### Bug Fixes

* no-wrap option
([#153](#153))
([9ae49e9](9ae49e9))
* update notification includes -g flag for cli
([#150](#150))
([370bfc6](370bfc6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants