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

Fixes spaces in project name #176

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fixes spaces in project name #176

wants to merge 2 commits into from

Conversation

rin-st
Copy link
Member

@rin-st rin-st commented Dec 20, 2024

An error appears here only when used foundry with project name with spaces at the end. It throws because / (space + /) is interpreted as separation between arguments. All other variants " a", "a b" etc works as expected

So idea is to restrict space usage at the end of the project name and allow all others

I added a check when prompting for a name: you can't go the next steps until you have no spaces/tabs at the end of the project name. Also added project field reset for adventurers who will try to deceive cli and try to use something like yarn cli "space-at-the-end "

Screen.Recording.2024-12-20.at.17.22.43.mov

how it looks with the extension
image

Fixes #150

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.

spaces in repo name gives error
1 participant