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

Methods of RegExp and String #414

Merged

Conversation

dbritto-dev
Copy link
Contributor

No description provided.

@dbritto-dev
Copy link
Contributor Author

@joaquinelio yo creo que ahora ya esta bien

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

  • Si rescribis en lugar de usar las suggest, cuida los acentos que les agregué/quité
  • una de las correccione esta sin suggest porque el md con graves me lo impide
  • para mi es valido "retornar" como el "devolver" que sugerimos en todos lados, no se que piensan otros. Sugerimos consistencia pero no creo que esto confunda a nadie.
  • no creo que debas usarr lista en lugar de array, para quien solo sepa js lista no tiene entidad. No lo puse en suggest porque en el momento dudaba...

9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

dbritto-dev and others added 7 commits October 9, 2020 00:40
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
dbritto-dev and others added 12 commits October 9, 2020 08:42
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
dbritto-dev and others added 7 commits October 9, 2020 13:54
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@dbritto-dev
Copy link
Contributor Author

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

AAAHHHH!!!
¿¿¿Lo pase al ingles con el signo "!" ????? sorrysorrysorrysorrysorry

linea 274
supongo que w+ es solamente "world"

ya está, me quemé.
perdon si los arrastre a uds tambien...

9-regular-expressions/17-regexp-methods/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@dbritto-dev
Copy link
Contributor Author

/done

Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Ok

@joaquinelio joaquinelio merged commit cab278a into javascript-tutorial:master Oct 12, 2020
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants