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

Closes: #1 #35082

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mimodbland
Copy link

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py

Fixes #35081

  • Tests pass
  • Appropriate changes to README are included in PR

I have carried out various checks of the new regular expression, in the regex101 web tool, using various possible uses. All the results have been correct.

blue.kale #True
.blue.kale #False
red_quinoa #True
red_quinoa #False
Alfa.norte #True
*hola #False
/ #False
Z? #False
A
- #False
B1_./ #False

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py
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.

Is modifying the regular expression the most feasible solution?
1 participant