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

@dbritto-dev dbritto-dev commented Oct 9, 2020

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Oct 9, 2020
@javascript-translate-bot javascript-translate-bot mentioned this pull request Oct 9, 2020
159 of 174 tasks complete
@dbritto-dev
Copy link
Contributor Author

@dbritto-dev dbritto-dev commented Oct 9, 2020

@joaquinelio yo creo que ahora ya esta bien

Copy link
Contributor

@joaquinelio joaquinelio left a comment

  • 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 Oct 9, 2020
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 Oct 9, 2020
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 6 commits Oct 9, 2020
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

@dbritto-dev dbritto-dev commented Oct 9, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Oct 9, 2020
Copy link
Contributor

@joaquinelio joaquinelio left a comment

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

@dbritto-dev dbritto-dev commented Oct 9, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Oct 9, 2020
Copy link
Contributor

@vplentinax vplentinax left a comment

Ok

@joaquinelio joaquinelio merged commit cab278a into javascript-tutorial:master Oct 12, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@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
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.