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

[p5.js - es] Translation of steward_guidelines.md #6894

Merged
merged 6 commits into from
Mar 19, 2024

Conversation

EmilioOcelotl
Copy link
Contributor

@EmilioOcelotl EmilioOcelotl commented Mar 19, 2024

Changes:

Translation of Steward Guidelines into Spanish.

Copy link

welcome bot commented Mar 19, 2024

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

@EmilioOcelotl EmilioOcelotl changed the title [p5.js - es] Translation of contributor_guidelines.md [p5.js - es] Translation of steward_guidelines.md Mar 19, 2024
@Guirdo Guirdo self-assigned this Mar 19, 2024
Copy link

@holomorfo holomorfo left a comment

Choose a reason for hiding this comment

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

---


## Tips y trucos

Choose a reason for hiding this comment

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

Para ser consistente con Marce, ella llamó a esta sección "Consejos y Trucos"

A veces, la cantidad de issues y PR que requieren revisión puede ser un poco abrumadora. Si bien intentamos implementar procesos que faciliten las cosas, hay algunos consejos y trucos que puedes utilizar para ayudar con la revisión de issues y PRs.


### Plantillas de respuesta

Choose a reason for hiding this comment

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

Para consistencia con marce
Plantillas de Respuesta

Copy link

@holomorfo holomorfo left a comment

Choose a reason for hiding this comment

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

Hola @EmilioOcelotl muchas gracias por tu texto te agrego los comentarios, principalmente:

  • itálicas para cualquier palabra en ingles en lugar de comillas, por favor mira el PR de Marce que es tu mismo documento para que veas como tiene su estandar [p5.js - es] Translate steward_guidelines #6873
  • Actualizar los nombres de las secciones para coincidir con el indicie de Marce

Con eso ya estoy listo para aprobar tu PR
CC: @Guirdo


### Plantillas de respuesta

Una característica útil de GitHub que puedes utilizar es la funcionalidad [Respuestas Hduardadas](https://docs.github.com/en/get-started/writing-on-github/working-with-saved-replies/about-saved-replies), que está disponible para usar al redactar una respuesta a issues o pull requests. Algunos de los flujos de trabajo descritos anteriormente pueden requerir responder a issues o PRs con respuestas idénticas o muy similares (redireccionar preguntas al foro, aceptar un problema para su corrección, etc.), y usar Respuestas Guardadas puede hacer que esto sea un poco más eficiente.

Choose a reason for hiding this comment

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

Respuestas Huduardadas? Guardadas supongo


Revisar un PR complejo puede ser difícil con comandos de git complejos necesarios para obtener la versión del código del PR localmente para que puedas probarla. Afortunadamente, [GitHub CLI](https://cli.github.com/) puede ayudar enormemente con este proceso y más.

Después de instalar GitHub CLI e iniciar sesión, revisar una PR localmente se puede hacer ejecutando el comando `gh pr checkout [id_del_pull_request]`, y el proceso de obtener un <em>fork</em> remoto, crear una rama y cambiar a la rama se realizan automáticamente para ti. Volver a la rama principal será lo mismo que cambiar de rama ejecutando `git checkout main`. ¡Incluso puedes dejar un comentario en el PR desde la CLI sin necesidad de visitar la página web en absoluto!

Choose a reason for hiding this comment

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

todas las palabras que se queden en ingles por favor ponerlas en itáliscas, esto aplica para todos los

  • fork
  • issues
  • pull requests

@Qianqianye Qianqianye added the Translation Translation of p5.js documentation label Mar 19, 2024
@EmilioOcelotl
Copy link
Contributor Author

Hice las modificaciones solicitadas.
También va el archivo webgl_contribution_guide.md con las convenciones comentadas

Copy link

@holomorfo holomorfo left a comment

Choose a reason for hiding this comment

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

Muchas gracias por las correcciones @EmilioOcelotl, aprobado.
CC: @Guirdo


### Tarea Variada

Todos los pasos pueden ejecutarse directamente con `npx grunt [paso]`. También hay algunas tareas que no se mencionan arriba pero podrían ser útiles en ciertos casos.
Copy link
Contributor

@Guirdo Guirdo Mar 19, 2024

Choose a reason for hiding this comment

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

Devolver npx grunt [paso] a su estado original, no se debe traducir el código


En lugar de monitorear manualmente las pestañas <em>Issues</em> o <em>Pull Requests</em> del repositorio en busca de nuevos <em>issues</em> o PRs, puedes "ver" el repositorio haciendo clic en el botón <em>Watch</em> con un ícono de ojo en la parte superior de la página del repositorio, frente al nombre del repositorio.

![Cropped screenshot of the top right corner of a GitHub repository page showing a series of buttons in the center from left to right: Sponsor, Watch, Fork, Starred.](images/github-repo-metrics.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Actualizar las rutas relativas de las imagenes

Suggested change
![Cropped screenshot of the top right corner of a GitHub repository page showing a series of buttons in the center from left to right: Sponsor, Watch, Fork, Starred.](images/github-repo-metrics.png)
![Cropped screenshot of the top right corner of a GitHub repository page showing a series of buttons in the center from left to right: Sponsor, Watch, Fork, Starred.](../images/github-repo-metrics.png)

## Recursos

- Lee nuestra [visión general de la arquitectura WebGL de p5.js](webgl_mode_architecture.md) para entender cómo difiere el modo WebGL del modo 2D. Esta será una referencia valiosa para algunos detalles de implementación para shaders, trazos y más.
- Lee nuestras [instrucciones para contribuyentes](https://p5js.org/contributor-docs/#/./contributor_guidelines) para obtener información sobre cómo crear issues, configurar el código base y probar cambios.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remplaza contribuyentes por contribuidores, ya que la primera es una palabra que se suele utilizar en otros ambitos como la economia y el derecho

@Guirdo
Copy link
Contributor

Guirdo commented Mar 19, 2024

El PR #6873 de @ml008008 ha sido fusionado, por favor haz pull de sus cambios e incorporalos a los tuyos

@EmilioOcelotl
Copy link
Contributor Author

Listo, quedo al pendiente.

Copy link
Contributor

@Guirdo Guirdo left a comment

Choose a reason for hiding this comment

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

Aprobado 💯! Muchas gracias, Emilio!

@Guirdo
Copy link
Contributor

Guirdo commented Mar 19, 2024

@Qianqianye @limzykenneth This PR is done!

@Qianqianye Qianqianye merged commit e403378 into processing:main Mar 19, 2024
2 checks passed
@Qianqianye
Copy link
Contributor

Qianqianye commented Mar 19, 2024

Looks great, thanks @EmilioOcelotl @Guirdo @holomorfo 💗
@all-contributors please add @EmilioOcelotl for translation

Copy link
Contributor

@Qianqianye

I've put up a pull request to add @EmilioOcelotl! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Translation Translation of p5.js documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants