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

Allows passing size or custom alphabet via cli as args #334

Merged
merged 2 commits into from Jan 16, 2022

Conversation

vitalybaev
Copy link
Contributor

@vitalybaev vitalybaev commented Jan 15, 2022

Closes #275

I've implemented minimal args parsing logic to fit our needs and be as little as possible

bin/parseArgs.js Outdated Show resolved Hide resolved
@ai
Copy link
Owner

@ai ai commented Jan 15, 2022

We need to improve test coverage. You can check test coverage locally by running pnpm test.

@ai
Copy link
Owner

@ai ai commented Jan 15, 2022

We also need to fix Node.js 12 support for binary (we can’t use ??)

@vitalybaev
Copy link
Contributor Author

@vitalybaev vitalybaev commented Jan 16, 2022

Done

I've faced error ERROR dual-publish error: Rename bin/utils.js to bin/utils/index.js, so I moved file according to this message.

Now it still works with Node 12 and test coverage is 100%

@ai ai merged commit 32b9bda into ai:main Jan 16, 2022
6 checks passed
@vitalybaev vitalybaev deleted the feature/cli-args branch Jan 16, 2022
@ai
Copy link
Owner

@ai ai commented Jan 16, 2022

bin/utils will create unnecessary files because of dual-publish. I cleaned up arguments parsing a little 27eaa90

And released in 3.2.

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.

2 participants