Skip to content

Replaced optimist with yargs #1093

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

Merged
merged 8 commits into from
Aug 7, 2020
Merged

Replaced optimist with yargs #1093

merged 8 commits into from
Aug 7, 2020

Conversation

Ratstail91
Copy link
Contributor

This is, admittedly just a drop-in replacement, but this issue pops up in literally every project I have, and it's driving me nuts.

@Ratstail91 Ratstail91 changed the title Replaced optimist with yargs (#1072) Replaced optimist with yargs Jul 24, 2020
@kibertoad
Copy link
Contributor

Looks good, thanks a lot! Could you address a couple of very minor comments?

@Ratstail91
Copy link
Contributor Author

Sure! I've addressed the comments. I can remove the redundant testing if needed, otherwise she's good to go!

@kibertoad
Copy link
Contributor

@Ratstail91 Could you please remove the GA part of the PR? I can remove it myself later, but won't have time for this today.

@Ratstail91
Copy link
Contributor Author

@kibertoad All done! Sorry for the delay!

@kibertoad
Copy link
Contributor

Thanks a lot!

@kibertoad kibertoad merged commit 877d93a into foreversd:master Aug 7, 2020
@hc-12
Copy link

hc-12 commented Aug 14, 2020

Thanks for the fix guys, can we have this as patch release soon?

@kibertoad
Copy link
Contributor

kibertoad commented Aug 22, 2020

@hc-12 Sorry it took a while.

Released in 3.0.1

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.

None yet

3 participants