Skip to content
This repository was archived by the owner on May 12, 2020. It is now read-only.

Temporarily fix https://www.npmjs.com/advisories/803 - #206

Merged
bsclifton merged 2 commits into
masterfrom
bsc-audit-deps
Apr 12, 2019
Merged

bsclifton merged 2 commits into
masterfrom
bsc-audit-deps

Conversation

@bsclifton

Copy link
Copy Markdown
Member

Once node-gyp issues a release, we can back this out and update. The version number for that will likely be 3.8.1

More info at: nodejs/node-gyp#1713

@bsclifton
bsclifton requested review from bbondy and diracdeltas April 12, 2019 06:04
@bsclifton bsclifton self-assigned this Apr 12, 2019
@bsclifton

Copy link
Copy Markdown
Member Author

NOTE: looks like CI is failing; will look at this

bsclifton added a commit that referenced this pull request Apr 12, 2019
- Backs out #206
- Updates node-gyp to official build which fixes https://www.npmjs.com/advisories/803
Once `node-gyp` issues a release, we can back this out and update. The version number for that will likely be `3.8.1`

More info at: nodejs/node-gyp#1713
I believe this was missed as part of #203
@bsclifton

Copy link
Copy Markdown
Member Author

Fixed 2 malware lists check- there are remaining failures which were caused by #197

@bsclifton
bsclifton merged commit bbb9790 into master Apr 12, 2019
@bsclifton
bsclifton deleted the bsc-audit-deps branch April 12, 2019 06:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants