Skip to content
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

npm upgrade; removing ejs and external template files #1046

Merged
merged 1 commit into from Feb 19, 2019
Merged

npm upgrade; removing ejs and external template files #1046

merged 1 commit into from Feb 19, 2019

Conversation

andygreenegrass
Copy link
Contributor

I ran an npm upgrade to reduce vulnerabilities in the audit.
The only outstanding vulnerabilities are from nyc and it requires a major version update. I didn't want to break any code coverage stuff, so I left it alone.

I also removed the dependency on external files in WSSecurityCert.ts, which also removed the need for ejs.

@jsdevel jsdevel merged commit 9aed566 into vpulim:master Feb 19, 2019
@jsdevel
Copy link
Collaborator

jsdevel commented Feb 19, 2019

i'm in awe @andygreenegrass . thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants