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

Remove optimist in favor of @author.io/arg #267

Merged
merged 1 commit into from
Mar 21, 2022

Conversation

forty
Copy link
Contributor

@forty forty commented Jul 29, 2020

This avoids a security issue on minimist, which is an indirect dependency

@forty forty force-pushed the forty/remove-optimist branch from 9e450b3 to 76f8a46 Compare July 29, 2020 09:12
@forty
Copy link
Contributor Author

forty commented Jul 29, 2020

Disclaimer: this was all done manually and I haven't tested anything :) please review with great care

@forty forty force-pushed the forty/remove-optimist branch 3 times, most recently from 5abd0d6 to cd34d47 Compare July 29, 2020 12:35
@forty forty changed the title Remove optimist in favor of @author.io/arg to avoid Remove optimist in favor of @author.io/arg Jul 29, 2020
This avoids a security issue on minimist, which is an indirect dependency
@aksswami
Copy link

aksswami commented Dec 9, 2021

@coreybutler possible to merge this anytime soon?

@JyotsnaC
Copy link

Looking for the same. Issue to be resolved.

@DFG-kjtroll
Copy link

Looking to deploy my express API with this, and my infrastructure guys will love to see a clean npm audit before global install... my current workaround is to force it to use minimist 0.2.1 using npm-force-resolutions. But that only patches a project install of node-windows and I'm not sure if it introduces bugs... Would love to see this merged!

@coreybutler coreybutler merged commit 2663a3e into coreybutler:master Mar 21, 2022
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.

5 participants