search

mikaelbr / node-notifier

A Node.js module for sending notifications on native Mac, Windows and Linux (or Growl as fallback)
MIT License
5.75k stars 322 forks source link

Corrected the way of importing uuid #329

Closed charlesnchr closed 4 years ago

charlesnchr commented 4 years ago

Deep requires of uuid have been deprecated, so I updated the import to get rid of DeprecationWarning.

charlesnchr commented 4 years ago

This is the warning it gets rid of: DeprecationWarning: Deep requiring like const uuidv4 = require('uuid/v4'); is deprecated as of uuid@7.x. Please require the top-level module when using the Node.js CommonJS module or use ECMAScript Modules when bundling for the browser. See https://github.com/uuidjs/uuid#deep-requires-now-deprecated for more information.