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

Add sugestion about regex #1758

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

Conversation

reloadercf
Copy link
Contributor

This is a problem of bootcamp, sometimes the students say: I don't understand my code, How I can continue with the next functions using replace because I don't know what is /[\s.,/#!$%^&*;:{}=-_`~()]/g,"" ?

@unjust
Copy link
Member

unjust commented Feb 27, 2024

@reloadercf bueno que intentas avanzar con eso - fue un conversación hace rato
#1637
Me gustaria si puedes tener un acuerdo con otros coaches y un review de ellos para cerrarlo.

Seria mejor si no es la primera cosa abajo de Javascript, porque los otros items son relacionado con OAs en particular.
Entonces con cual OA aparecen los regex? Debemos mencionarlo alla, o en un parte/seccion mas general.

@unjust unjust added the idea Ideas, sugerencias, comentarios generales y feedback label Feb 27, 2024
@unjust unjust linked an issue Feb 27, 2024 that may be closed by this pull request
Copy link
Contributor

@AlexisJoselyn AlexisJoselyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De acuerdo que las estudiantes tengan esto claro en el readme. En estos primero días de código de DEV014, el haber seguido la sugerencia de no usar regex las está empujando a abordar las solciones más desde la lógica.

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Limitamos el uso de Regex en Text Analyzer?
3 participants