search

yeoman / update-notifier

Update notifications for your CLI app
BSD 2-Clause "Simplified" License
1.76k stars 132 forks source link

Fix docs position of `shouldNotifyInNpmScript` #143

Closed renatoagds closed 6 years ago

renatoagds commented 6 years ago

Now, shouldNotifyInNpmScript is described as an notify option in the Documentation, but, looking at the source code it's an option from constructor: https://github.com/yeoman/update-notifier/blob/master/index.js#L43

Just move shouldNotifyInNpmScript for correct place in documentation.

renatoagds commented 6 years ago

@sindresorhus can you take a look? just updating docs 🤘