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

fix: print regex error #488

Merged
merged 1 commit into from Nov 7, 2021
Merged

fix: print regex error #488

merged 1 commit into from Nov 7, 2021

Conversation

MarcoIeni
Copy link
Contributor

@MarcoIeni MarcoIeni commented Nov 7, 2021

Before:
image

After:
image

@sassman
Copy link
Member

sassman commented Nov 7, 2021

nice, could you pls share the username that you have used to provoke this error? I'd like to reproduce it very identically.

@MarcoIeni
Copy link
Contributor Author

yes, sure:

[placeholders.username]
type = "string"
prompt = "GitHub username (or organization)?"
regex = "/^[a-z\\d](?:[a-z\\d]|-(?=[a-z\\d])){0,38}$/i"

@MarcoIeni
Copy link
Contributor Author

Probably I need to rename placeholders.username to placeholders.gh-username, because I see that {{username}} is now reserved :)

@sassman
Copy link
Member

sassman commented Nov 7, 2021

/rebase

@sassman sassman merged commit eaec0bf into cargo-generate:master Nov 7, 2021
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

2 participants