Skip to content

Small grammar corrections #292

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

Merged
merged 2 commits into from
Oct 7, 2019
Merged

Small grammar corrections #292

merged 2 commits into from
Oct 7, 2019

Conversation

elchininet
Copy link
Contributor

Small grammar corrections and link to Create React App repository added

@netlify
Copy link

netlify bot commented Oct 3, 2019

Deploy preview for es-reactjs ready!

Built with commit 3df61a8

https://deploy-preview-292--es-reactjs.netlify.com

Copy link
Member

@carburo carburo left a comment

Choose a reason for hiding this comment

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

¡Hola, @elchininet! Muchas gracias por las correciones, solo tengo dos comentarios sobre algunos cambios que creo que podrían alterar la traducción del original. Es importante para nosotros mantener la sincronización con el repositorio en inglés (incluyendo los números de línea), porque de esa forma se facilitan las actualizaciones periódicas que hacemos con ayuda de un bot.

@elchininet
Copy link
Contributor Author

elchininet commented Oct 5, 2019

Hola Rainier, en mi opinión, esta corrección es para mantener coherencia con el texto en español más que con el de inglés. Más abajo el título de la sección y la mención al plugin se realiza con la palabra ESLint, no con el sustantivo en inglés linter, es por eso que pensé que debería ser lo mismo en este lugar. De todas maneras dejo en tus manos decidir cuál crees que es el más correcto en este caso. Un saludo.

Co-Authored-By: Rainer Martínez Fraga <[email protected]>
@carburo carburo merged commit 66b18eb into reactjs:master Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants