-
Notifications
You must be signed in to change notification settings - Fork 820
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
Ran npm audit fix
to update package dependencies.
#1097
Conversation
Build Succeeded 👏 Build Id: 4cf80c0c-bc1b-48f8-91ca-c96a1888b7c7 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
e4a7674
to
e7e1a50
Compare
Build Succeeded 👏 Build Id: 91e6b19e-d41b-4ec5-87cf-f1acc3da035f The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
LGTM however I wonder what's using handlebars - perhaps nyc for html display of coverage? |
Build Succeeded 👏 Build Id: 30203f3d-dc26-47ac-b843-056bfe360485 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: pooneh-m, roberthbailey The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I have no idea what's using it. Feel free to investigate & cull once you are back at work. In the mean time, I'll apply the update to close the security hole. |
@steven-supersolid - While running lint and test on my other PR npm notified me that I had dependencies with security vulnerabilities. This PR updates dependencies to address them.