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

Update node-watch to 0.7.1 #1524

Closed
3 tasks done
Krinkle opened this issue Dec 7, 2020 · 1 comment · Fixed by #1534
Closed
3 tasks done

Update node-watch to 0.7.1 #1524

Krinkle opened this issue Dec 7, 2020 · 1 comment · Fixed by #1534
Assignees
Labels
Category: Release Type: Enhancement New idea or feature request.

Comments

@Krinkle
Copy link
Member

Krinkle commented Dec 7, 2020

Update node-watch from 0.7.0 to 0.7.1 (latest).

https://github.com/yuanchuan/node-watch/releases/tag/0.7.1
https://github.com/yuanchuan/node-watch/commits/master

  • Verify npm publisher.
  • Review package diffs.
  • Stage update in PR and confirm releated features: CLI file watching.

Example at #1522.

@Krinkle Krinkle added Type: Enhancement New idea or feature request. Category: Release labels Dec 7, 2020
@Krinkle Krinkle self-assigned this Dec 7, 2020
@Krinkle
Copy link
Member Author

Krinkle commented Dec 31, 2020

Verify publication: node-watch 0.7.0 -> 0.7.1

npm info node-watch@0.7.0
$ node-watch@0.7.0 | MIT | deps: none

published 2 months ago by yuanchuan <yuanchuan23@gmail.com>
$ npm info node-watch@0.7.1
node-watch@0.7.1 | MIT | deps: none

maintainers:
- yuanchuan <yuanchuan23@gmail.com>
- intervalia <intervalia@gmail.com>

published 3 weeks ago by yuanchuan <yuanchuan23@gmail.com>
  • ✅ It was published by the same account as previously.
  • ⚠️ 1 new maintainer(s) were added.
  • ✅ No new dependencies were added.
  • ✅ No dependencies changed.

@intervalia was added as maintainer with npm publishing rights. This is confirmed from a previously trusted account at yuanchuan/node-watch#63 (comment).

LGTM.

Review package diff: node-watch 0.7.0 -> 0.7.1

https://diff.intrinsic.com/node-watch/0.7.0/0.7.1

  • Minor changes relating to Windows support. The changes appear well-documented and well-tested by upstream.
  • Node 8 is no longer integrated by upstream CI. QUnit has started to require Node 10 relatively recently, so this is no longer a concern.

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Release Type: Enhancement New idea or feature request.
Development

Successfully merging a pull request may close this issue.

1 participant