Skip to content

Conversation

@jameshy
Copy link
Owner

@jameshy jameshy commented Apr 6, 2021

Changes:

  • upgrade all dependencies in package.json
  • remove through2 and use built in Node.js functions

waiting for upstream package nyc to upgrade yargs dependency, then the security advisory in y18n will be fixed.

the through2 library is not required, it's functionality is available in core nodejs
@jameshy jameshy added the blocked label Apr 6, 2021
@coveralls
Copy link

coveralls commented Apr 6, 2021

Coverage Status

Coverage remained the same at 92.414% when pulling 7934ea9 on upgrade-dependencies into e9fb1ef on master.

@jameshy
Copy link
Owner Author

jameshy commented Apr 25, 2021

istanbuljs/nyc#1402

@jameshy jameshy force-pushed the upgrade-dependencies branch from 9ca9bc4 to 7934ea9 Compare May 12, 2021 21:54
@jameshy jameshy merged commit 9adf955 into master May 12, 2021
@jameshy jameshy deleted the upgrade-dependencies branch December 10, 2022 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants