-
Notifications
You must be signed in to change notification settings - Fork 462
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
Limitamos el uso de Regex en Text Analyzer? #1637
Comments
Prohibamos el uso de ellas en el readme y quitemos cualquier recurso o referencia que hable sobre ellas, al menos en este proyecto |
Creo que se podría restringir en determinadas funcionalidades, por ejemplo en la que cuenta los números y sugerir que ahí usen isNaN de forma explicita, regex les puede ser útil en el futuro pero restringirlo puede darles la idea que no es útil o que no deben usarlo. |
No lo veo un lugar donde hablamos de regex o compartimos recursos sobre eso. Seria bueno saber, en cual parte estan usando regex - para que especifica requisito? Y que estan reemplazando con el regex - un bucle o en algun metodo como |
Estamos viendo que muchas usan regex en lugar de bucles.
Deberiamos poner algo en el readme ?
The text was updated successfully, but these errors were encountered: