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

Adds translations directory, Spanish README -- linked in original README #737

Merged
merged 1 commit into from Sep 10, 2020

Conversation

ari-os310
Copy link
Contributor

Does not 'resolve' issue #726, but I had commented to @ljharb about adding a Spanish translation for readme

I think I have properly linked in the original readme. If anything is not best practice, I'm happy to fix

@ljharb
Copy link
Member

ljharb commented Aug 30, 2020

@ari-os310 thanks! my intention is that the Spanish translation would be in your master branch, and that this PR would only contain b798c68, but would link to your forked repo instead of leaving the translation inlined in this repo.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks! Translation seems good as well, to my novice eye.

@ljharb ljharb merged commit f702f62 into jsx-eslint:master Sep 10, 2020
@ari-os310
Copy link
Contributor Author

Thank you @ljharb !

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

2 participants