Skip to content

Addresses audit advisory in production dependencies. #39

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qubyte
Copy link
Contributor

@qubyte qubyte commented May 8, 2021

Optimist was bringing this in: https://npmjs.com/advisories/1179

This PR replaces optimist (deprecated) with yargs (recommended successor with same API and without the issue).

Optimist was bringing this in: https://npmjs.com/advisories/1179
This PR replaces optimist (deprecated) with yargs (successor with
same API and without the issue).
@vercel
Copy link

vercel bot commented May 8, 2021

Someone is attempting to deploy a commit to a Personal Account owned by @remy on Vercel.

@remy first needs to authorize it.

@sheldonhull
Copy link

bump for @remy

@paulrobertlloyd
Copy link

Anything preventing this from getting merged and released?

@remy
Copy link
Owner

remy commented Jan 19, 2022

Just that I hadn't tested the compatibility.

I think I spotted an issue with the service that keeps the github tokens too - so I need to do a bit of maintenance on wm, and I'll get on with merging this too (just on mobile ATM)

@qubyte
Copy link
Contributor Author

qubyte commented Jan 19, 2022

Let me know if you have issues. I’m happy to update this PR.

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

Successfully merging this pull request may close these issues.

4 participants