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

Update minimist to latest #13

Closed
hkuchibhotla opened this issue Sep 2, 2020 · 5 comments
Closed

Update minimist to latest #13

hkuchibhotla opened this issue Sep 2, 2020 · 5 comments

Comments

@hkuchibhotla
Copy link
Contributor

hkuchibhotla commented Sep 2, 2020

This repo uses minmist 0.0.7, which seems to have some security vulnerabilities.
Can we update to the latest 1.2.5

   "minimist": "1.2.5"

Tests Passed

@ungoldman
Copy link
Collaborator

fine by me, PR welcome

@hkuchibhotla
Copy link
Contributor Author

hkuchibhotla commented Sep 3, 2020

@ungoldman Thank you, can i get write access to commit/ open a PR?

@ungoldman
Copy link
Collaborator

ungoldman commented Sep 3, 2020

@hkuchibhotla you can fork this repo and open a PR by following these instructions: https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork

@hkuchibhotla
Copy link
Contributor Author

Thank you @ungoldman, here it is: #14

@ungoldman
Copy link
Collaborator

ungoldman commented Sep 10, 2020

Thank you @hkuchibhotla! Resolved in #14, published in v1.0.4

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

No branches or pull requests

2 participants