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

Warns the user on providing multiple arguments for <projectName> #24

Merged

Conversation

jamesgeorge007
Copy link
Contributor

Closes #23

Currently, create-proton-app <arg1> <arg2> works fine without any sort of warnings that doesn't make any sense.

This PR fixes it so that the user is warned appropriately.

Copy link
Owner

@albe-rosado albe-rosado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, so much for your time, I totally agree, can you instead of using the chalk library, use the ansiColors defined in the utils module? I didn't want to include another library just for the sake of printing colors in the terminal, trying to keep the projects with as few deps as possible. Again, thank you.

@jamesgeorge007 jamesgeorge007 changed the title Warns the user if he provides multiple arguments Warns the user on providing multiple arguments for <projectName> May 12, 2019
src/createApp.js Outdated Show resolved Hide resolved
@albe-rosado albe-rosado merged commit 82fcc08 into albe-rosado:master May 14, 2019
@albe-rosado
Copy link
Owner

Thank you!

@jamesgeorge007 jamesgeorge007 deleted the hotfix/validate-arguments branch January 2, 2021 14:23
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.

Users can fire in any number of arguments as the project-name
2 participants