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

Keyword pattern #63

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

Conversation

GarciaBarreiro
Copy link

Before, the keyword pattern only accepted words with letters [A-Za-z] and vowels with acute accents (only in the middle/end of the word, not at the beggining, due to how \h works).

This behaviour doesn't accept other languages written in latin alphabet, such as spanish (needs to accept ñ), or french (needs to accept ç, vowels with different kinds of accents).

This solution isn't ideal, as I think it ends up accepting more characters than needed (and also looking quite ugly), but I tested it and I don't notice any downside in its use.

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.

1 participant