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

Limitamos el uso de Regex en Text Analyzer? #1637

Open
unjust opened this issue Nov 10, 2023 · 3 comments · May be fixed by #1758
Open

Limitamos el uso de Regex en Text Analyzer? #1637

unjust opened this issue Nov 10, 2023 · 3 comments · May be fixed by #1758
Assignees
Labels
idea Ideas, sugerencias, comentarios generales y feedback question Further information is requested

Comments

@unjust
Copy link
Member

unjust commented Nov 10, 2023

Estamos viendo que muchas usan regex en lugar de bucles.
Deberiamos poner algo en el readme ?

@unjust unjust added the idea Ideas, sugerencias, comentarios generales y feedback label Nov 10, 2023
@JuanCarlosHM
Copy link
Contributor

Prohibamos el uso de ellas en el readme y quitemos cualquier recurso o referencia que hable sobre ellas, al menos en este proyecto

@jmontes50
Copy link
Contributor

jmontes50 commented Nov 10, 2023

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.

@unjust unjust added this to the 7.5 Next Release milestone Nov 14, 2023
@unjust
Copy link
Member Author

unjust commented Nov 14, 2023

No lo veo un lugar donde hablamos de regex o compartimos recursos sobre eso.
podemos meter una pauta si habia una seccion como "consideraciones tecnicas"

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 getCharacterCountExcludingSpaces?
Ahi cuando mencionamos el requisito podemos decir que recomendamos que no usen regex.

@unjust unjust added the question Further information is requested label Nov 15, 2023
@unjust unjust removed this from the 7.6 Next Release milestone Dec 20, 2023
@unjust unjust linked a pull request Feb 27, 2024 that will close this issue
@unjust unjust linked a pull request Feb 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea Ideas, sugerencias, comentarios generales y feedback question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants