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

Translated Docs to Spanish #65

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LuisPaGarcia
Copy link

No description provided.

Copy link

@CicherMK CicherMK left a comment

Choose a reason for hiding this comment

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

The translation looks ok 🤓👍
(la traducción se ve muy bien)

Copy link

@Charly6596 Charly6596 left a comment

Choose a reason for hiding this comment

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

Looks good overall, but felt like there is inconsistency with the usage of "tú" and "usted", and found some things that, in my opinion, could be improved

There might be some comments related to country differences (I deleted a few of those tho), since I'm a Spaniard and this looks written by someone from Lat. America, I mean, there is nothing wrong with that, but it's that some expressions I corrected might be right, but they just sound weird to me.

Please let me know what you think about my comments, and I am of course open to any kind of feedback (:

README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
README-es-spa.md Outdated Show resolved Hide resolved
@LuisPaGarcia
Copy link
Author

Thank you for your comments, @Charly6596 . Already pushed the requested changes.

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