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
fix(lang): fix "Not enough non-option arguments" message for the Czech language#2242
Conversation
Removing an extra comma at the end of the line.
It seems that an additional formal step is still required. What is it? Can I help with it? |
It's a very trivial issue (only a typo, in fact) and simple to review. Why it takes so much time to merge? |
@mlc-mlapis It needs to be approved by some maintainer: review is required by reviewers with write access. Maybe @bcoe may help, please? The latest 17.6.0 release is unusable. |
@xmedeko Thanks for the help. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verified that this fixes the problem. Thanks for the contribution!
Bug introduced in #2220 |
@bcoe I'm not familiar with the tag/release protocol of yargs anymore, otherwise I would squash and merge this to prepare a 17.6.1 patch release. Can we get a little help here, since this is affecting |
Since this was not merged yet (a typo, in 25 days, really?) and it's breaking the nx package, I've found the following workaround. It may help someone. Run node process with env var |
I'll go ahead and merge it. Will have to figure out how to get a 17.6.1 release published. Yargs is used by thousands of projects, and is (or at least was) used as a canary-in-the-goldmine package to help verify regressions of Node itself, so releases carry a lot of weight. Perhaps @bcoe has been able to automate everything via release-please. Will see. |
Removing an extra comma at the end of the line.