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

Updating Spanish and Fixing MD violations #3563

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stonkol
Copy link

@stonkol stonkol commented Dec 28, 2023

Updating Spanish and Fixing MD violations in CONTRIBUTING.md

Copy link

vercel bot commented Dec 28, 2023

@stonkol is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added documentation Improvements or additions to documentation. doc-translation README doc translations. labels Dec 28, 2023
@stonkol stonkol changed the title Updating Spanish and Fixing MD violations in CONTRIBUTING.md Updating Spanish and Fixing MD violations Dec 28, 2023
@@ -379,9 +379,9 @@ Por lo general, no podrás acomodar las imágenes una al lado de la otra. Para h

#### [Échale un vistazo a este tutorial paso a paso de @codeSTACKr](https://youtu.be/n6d4KHSKqGk?t=107)

Desde que la API de GitHub permite solo 5k peticiones por hora, es posible que mi `https://github-readme-stats.vercel.app/api` pueda llegar al límite. Si lo alojas en tu propio servidor de Vercel, no tendrás que preocuparte de nada. ¡Clickea en el botón "Deploy" para comenzar!
Desde que la API de GitHub permite solo 5k peticiones por hora, es posible que mi `https://github-readme-stats.vercel.app/api` pueda llegar al límite. Si lo alojas en tu propio servidor de Vercel, no tendrás que preocuparte de nada. ¡Presione en el botón "Deploy" para comenzar!

Choose a reason for hiding this comment

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

The rest of the document uses informal form. I think to keep it consistant this should too, so I think Presiona would be better instead.

- [Excluir repositorios individualmente](#excluir-repositorios-individualmente)
- [Ocultar lenguajes individualmente](#ocultar-lenguajes-individualmente)
- [Mostrar más lenguajes](#mostrar-más-lenguajes)
- [Diseño Compacto de Tarjeta de Lenguaje](#diseño-compacto-de-tarjeta-de-lenguaje)

Choose a reason for hiding this comment

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

I think Compacto, Tarjeta and Lenguaje should not be in capital case

@@ -156,13 +156,13 @@ Puedes personalizar el aspecto de tu `Tarjeta de Estadísticas` o `Tarjeta de Re
- `icon_color` - Color de icono si esta disponible _(hex color)_
- `bg_color` - Color de fondo _(hex color)_
- `hide_border` - Oculta el borde de la tarjeta _(booleano)_
- `theme` - Nombre del tema, elige uno de [todos los temas disponible ](../themes/README.md)
- `theme` - Nombre del tema, elige uno de [todos los temas disponible](../themes/README.md)

Choose a reason for hiding this comment

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

This sentence is in plural. I think the correct way would be todos los temas disponibles

Copy link
Collaborator

@qwerty541 qwerty541 left a comment

Choose a reason for hiding this comment

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

Hey, @stonkol and @danicruz0415. Thanks for your interest in our project and your efforts. In nearest future we're going either to remove documentation translations from repository or migrate on automatic translations. We're making this decision for these reasons:

  • Most of documentation translations are outdated and hardly someone will completely up to date them. Even if one of documents will be updated, most still confuses users with oudated information.
  • Our team cannot properly review pull requests with translations since we don't know all these languages.
  • Removing translations will decrease overall efforts for project maintenance.

For these reasons currently we don't review documentation translations pull requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-translation README doc translations. documentation Improvements or additions to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants