[p5.js - es] Translate steward_guidelines #6873
Conversation
#NEW FILE , ADDING COMMENT FOR TESTING PURPOSE.
Starting translations
Translate steward_guidelines
|
Translate steward_guidelines |
Informe de errores (Bug Report issues)
Feature request
Feature enhancement
Discusión
Ready for review
|
Ready for review. |
Dependabot
There was a problem hiding this comment.
Hola Marce, muchas gracias por tu traducción. Te agrego algunos cambios que son necesarios y otros que son sugerencias, por favor revisa y actualiza donde sea necesario. Los más importantes son estos:
- Cualquier palabra que se quede en inglés debe estar en itálicas o entre comillas, y después entre paréntesis donde sea necesario agregar la traducción.
- Por favor traducir el nombre de los enlaces.
- Por favor elegir un estilo y dejarlo consistente en cuanto al uso de paréntesis y espacio, Por ejemplo:
-
este texto tiene un parentesis (aqui deje espacio, pero al cerrar no). Aquí continua el párrafo.
-
- Esto para evitar situaciones como esta:
-
Este es fragmento(continuado con parentesis ).Sigue la oración...
-
- Los títulos de este texto coinciden mucho con el texto de Diana, por favor revisar para usar los mismos o muy similares.
Por favor revisa y dime si te parecen adecuadas las correcciones o hay algo que debamos platicar con el equipo.
CC: @Guirdo
| @@ -0,0 +1,384 @@ | |||
| # Directrices para Administradores | |||
There was a problem hiding this comment.
queda pendiente al decisión final para Stewards
| # Directrices para Administradores | ||
|
|
||
|
|
||
| Ya sea que recién te hayas unido a nosotros como administrador, seas un mantenedor experimentado de p5.js, o estés en algún punto intermedio, esta guía contiene información, así como consejos y trucos que te ayudarán a contribuir de manera efectiva a p5.js. La mayor parte de lo que se escribe aquí son pautas, a menos que se indique lo contrario, lo que significa que puedes adaptar las prácticas mostradas aquí para que se ajusten a tu flujo de trabajo. |
There was a problem hiding this comment.
la palabra "mantenedor" no me parece adecuada sugiero algo como lo siguiente:
Ya sea que recién te hayas unido a nosotros como administrador, tengas experiencia previa en el mantenimiento de P5.js
There was a problem hiding this comment.
@ml008008 no se ve el cambio que te sugirio @holomorfo en este nuevo commit, sigue como "mantenedor"
There was a problem hiding this comment.
No se, si aplique el cambio. Lo envio de nuevo.
There was a problem hiding this comment.
probablemente no se agregó en el commit por equivocación
| ## Tabla de Contenidos | ||
|
|
||
| - [Issues](steward_guidelines.md#issues) | ||
| - [Bug report](steward_guidelines.md#bug-report) |
There was a problem hiding this comment.
Falta traducir la tabla, por favor revisa el PR de Diana con algunos ejemplos de como podemos dejar la palabra en ingles pero incluir la traducción y significado para quien no conozca los terminos.
https://github.com/processing/p5.js/pull/6876/files
There was a problem hiding this comment.
#Agrege un comentario nuevo para revisiøn sobre esta solicitud.
There was a problem hiding this comment.
Los titulos en la tabla de contenidos hacen referencia a temas o actividades propias de trabajar en un repositorio, por lo que si deberian ser traducidos aplicando la misma regla de dejar las palabras de la interfaz de github sin traducir y otras que también nos habiamos puesto de acuerdo en no traducir.
There was a problem hiding this comment.
Me podrian ayudar con la lista de palabras de interfaz de github, por favor? Para confirmar, porque para mi casi todo el contenido de la tabla que me toco aplica.
|
|
||
| ## Issues | ||
|
|
||
| Alentamos a la mayoría de las contribuciones de código fuente a comenzar con un problema, y como tal, los issues son el lugar donde la mayoría de las discusiones tendrán lugar. Los pasos a seguir al revisar un problema dependerán del tipo de problema que sea. El repositorio utiliza [GitHub issue templates] (https://github.com/processing/p5.js/blob/main/.github/ISSUE_TEMPLATE) para organizar mejor los diferentes tipos de problemas y alentar a los autores de problemas a proporcionar toda la información relevante sobre sus problemas. El primer paso al revisar el issue a menudo será revisar la plantilla completada y determinar si necesita información adicional por ejemplo, porque algunos campos no se completaron o se utilizó la plantilla incorrecta. |
There was a problem hiding this comment.
traducir el texto del link de Github por favor
There was a problem hiding this comment.
Siempre que se deje una palabra en inglés como Issue por favor ponerlo en itálicas
There was a problem hiding this comment.
itálicas aplicadas.
| - Si su entorno de prueba difiere de lo que se informa en el problema(issue) (por ejemplo, un navegador o sistema operativo diferente)(e.g., a different browser or OS): | ||
| - Deje un comentario diciendo que no puede replicar en su entorno específico. | ||
| - Agregue una etiqueta `help wanted` al problema(issue) y pida a alguien más con la configuración especificada en el problema que intente replicar el error. | ||
| - Sometimes, bugs only occur when using the web editor and not when testing locally. In this case, the issue should be redirected to the |
| - [ ] El fix no debe tener ningún impacto en la accesibilidad de p5.js. | ||
| - [ ] El fix debe utilizar el estándar moderno de codificación en JavaScript. | ||
| - [ ] El fix debe pasar todas las pruebas automatizadas e incluir nuevas pruebas(tests) si son relevantes. | ||
| 4. Si se requieren cambios adicionales, se deben agregar comentarios en línea a las líneas relevantes según se describió anteriormente [here](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request). |
| - [ ] El fix debe pasar todas las pruebas automatizadas e incluir nuevas pruebas(tests) si son relevantes. | ||
| 4. Si se requieren cambios adicionales, se deben agregar comentarios en línea a las líneas relevantes según se describió anteriormente [here](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request). | ||
| - Un bloque de sugerencias también puede ser usado para sugerir cambios específicos:\ | ||
| \ |
There was a problem hiding this comment.
me parece que hay que actualizar el path relativo para todas las imágenes
There was a problem hiding this comment.
Sugiero revisar antes de removerlo.
Justificaciøn: El path contiene los nombres de las carpetas que estan anidades y que apuntan al archivo, imagen, etc.
There was a problem hiding this comment.
Para actualizar los rutas relativas de las imagenes, bastaria con con subieras un nivel, de esta forma.
| \ | |
| \ |
Y así para el resto de las imagenes
There was a problem hiding this comment.
De acuerdo con sebastian, por favor actualiza el path a como dice él con "../" para todas, ya que no se están dibujando
| \ | ||
| \ | ||
|  | ||
| - Si se requieren múltiples cambios, no agregues comentarios de una sola línea muchas veces. En su lugar, sigue el procedimiento documentado [here](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request) para hacer comentarios de varias líneas y una sola solicitud de cambios (change request). |
|
|
||
| Las PRs de Dependabot generalmente solo son visibles para los administradores del repositorio, así que si esto no aplica a ti, por favor omite esta sección. | ||
|
|
||
| - Las PRs de Dependabot pueden fusionarse(be merged) directamente si la actualización de la versión es una [semver](https://semver.org/) versión de parche( patch version) y la prueba automatizada de CI(CI test ) ha pasado. |
There was a problem hiding this comment.
los paréntesis estaban incluidos en el texto original. Considerando esto, los borramos?
| Las PRs de Dependabot generalmente solo son visibles para los administradores del repositorio, así que si esto no aplica a ti, por favor omite esta sección. | ||
|
|
||
| - Las PRs de Dependabot pueden fusionarse(be merged) directamente si la actualización de la versión es una [semver](https://semver.org/) versión de parche( patch version) y la prueba automatizada de CI(CI test ) ha pasado. | ||
| - Las PRs de Dependabot con cambios de versión semver menor generalmente se pueden fusionar directamente siempre y cuando la prueba automatizada de CI pase. Se recomienda hacer una rápida verificación en el registro de cambios(changelog ) de la dependencia actualizada. |
There was a problem hiding this comment.
los paréntesis estaban incluidos en el texto original. Considerando esto, los borramos?
|
@holomorfo Entendido trabajare en las correcciones y sugerencias mencionadas. Gracias! |
Sugerencias implementadas y comentarios nuevos.
|
|
||
| ## Issues | ||
|
|
||
| Alentamos a la mayoría de las contribuciones de código fuente a comenzar con un issue, y como tal, los "issues" son el lugar donde la mayoría de las discusiones tendrán lugar. Los pasos a seguir al revisar un issue dependerán del tipo de issue que sea. El repositorio utiliza Plantillas de issues de GitHub (https://github.com/processing/p5.js/blob/main/.github/ISSUE_TEMPLATE) para organizar mejor los diferentes tipos de problemas y alentar a los autores de problemas a proporcionar toda la información relevante sobre sus problemas. El primer paso al revisar el "issue" a menudo será revisar la plantilla completada y determinar si necesita información adicional por ejemplo, porque algunos campos no se completaron o se utilizó la plantilla incorrecta. |
There was a problem hiding this comment.
Aquí se ha roto el link, por lo que debes poner el texto del enlace entre corchetes cuadrados
[Plantillas de issues de GitHub](https://github.com/processing/p5.js/blob/main/.github/ISSUE_TEMPLATE)
There was a problem hiding this comment.
"Issue" no deberia ser traducido en este parrafo.
- "los diferentes tipos de issues"
- "los autores de issues"
|
|
||
| ### "Bug report"(informe de error) | ||
|
|
||
| Los "Bug report issues" (informes de errores de issues) deberían utilizar la plantilla de Issue "Found a bug". El siguiente flujo de trabajo es típico para abordar los informes de errores: |
There was a problem hiding this comment.
En este caso los issues son para el informe de errores
| Los "Bug report issues" (informes de errores de issues) deberían utilizar la plantilla de Issue "Found a bug". El siguiente flujo de trabajo es típico para abordar los informes de errores: | |
| Los issues de informes de errores deberían utilizar la plantilla de Issue "Found a bug". El siguiente flujo de trabajo es típico para abordar los informes de errores: |
|
|
||
|
|
||
|
|
||
| ### "Feature request" (solicitud de función) |
There was a problem hiding this comment.
Para traducir "feature" es mejor utulizar la palabra "funcionalidad", pues funcionalidad es la capacidad de un sistema para cumplir con su objetivo o una tarea en especifico
| - ¿Es probable que la función(feature) propuesta cause un cambio incompatible? | ||
| - ¿Entrará en conflicto con las funciones y variables existentes de p5.js? | ||
| - ¿Entrará en conflicto con los bocetos típicos(typical sketches) ya escritos para p5.js? | ||
| - Las funciones que probablemente causen conflictos, como las mencionadas anteriormente, se consideran cambios incompatibles. Sin un lanzamiento de [Lanzamiento de versión mayor](https://docs.npmjs.com/about-semantic-versioning), no deberíamos realizar cambios incompatibles en p5.js. |
There was a problem hiding this comment.
El primer "lanzamiento de" esta demás
| - Las funciones que probablemente causen conflictos, como las mencionadas anteriormente, se consideran cambios incompatibles. Sin un lanzamiento de [Lanzamiento de versión mayor](https://docs.npmjs.com/about-semantic-versioning), no deberíamos realizar cambios incompatibles en p5.js. | |
| - Las funciones que probablemente causen conflictos, como las mencionadas anteriormente, se consideran cambios incompatibles. Sin un[Lanzamiento de versión mayor](https://docs.npmjs.com/about-semantic-versioning), no deberíamos realizar cambios incompatibles en p5.js. |
There was a problem hiding this comment.
falta espacio entre "un" y "Lanzamiento"
| Este tipo de issue tiene una plantilla mínima de discusión y debería ser utilizada para recopilar comentarios y retroalimentaciones sobre un tema en general antes de consolidarlo en algo más específico, como una solicitud de función. Estos "issues" de discusión pueden cerrarse cuando finaliza la conversación y se han creado los "issues" más específicos resultantes: | ||
|
|
||
| - Si se abre un "issue" como una discusión pero debería ser, por ejemplo, un "bug report"(informe de error), se debe aplicar la etiqueta correcta y quitar la etiqueta de "discusión". Además, se debe solicitar información adicional sobre el error al autor si aún no se ha incluido. | ||
| - Si se abre un issue como una discusión pero no es relevante para la contribución de código fuente o de otra manera relevante para los repositorios de GitHub/el proceso de contribución/la comunidad de contribución, deberían ser redirigidos al foro o a Discord y el problema cerrado. |
There was a problem hiding this comment.
Sugiero remplazar las diagonales "/" por comas e enumerar, de la siguente forma:
| - Si se abre un issue como una discusión pero no es relevante para la contribución de código fuente o de otra manera relevante para los repositorios de GitHub/el proceso de contribución/la comunidad de contribución, deberían ser redirigidos al foro o a Discord y el problema cerrado. | |
| - Si se abre un issue como una discusión pero no es relevante para la contribución de código fuente o de otra manera relevante para los repositorios de GitHub, el proceso de contribución o la comunidad de contribución, deberían ser redirigidos al foro o a Discord y el problema cerrado. |
| Casi todas las contribuciones de código a los repositorios de p5.js se realizan a través de Pull Request. Los administradores y los encargados de mantenimiento pueden tener "push access" (acceso de escritura) a los repositorios, pero aún se les anima a seguir el mismo proceso de issue > PR > proceso de revisión al contribuir con código. Aquí están los pasos para revisar una PR: | ||
|
|
||
| - La plantilla de pull request se puede encontrar [Aquî](https://github.com/processing/p5.js/blob/main/.github/PULL_REQUEST_TEMPLATE.md). | ||
| - Casi todas las solicitudes de pull requests deben tener problemas asociados abiertos y discutidos primero, lo que significa que los relevantes ["issue workflow"](steward_guidelines.md#issues) deben haber sido seguidos primero antes de que una PR sea revisada por cualquier administrador o encargado de mantenimiento. |
There was a problem hiding this comment.
Sugiero quitar "las solicitudes", pues es redundante solicitar una "request".
|
|
||
| ### "Simple fix" (arreglo simple/soluciøn sencilla) | ||
|
|
||
| Los "simple fix", como la corrección de un pequeño error tipográfico, pueden "be merged" (fusionarse) directamente por cualquier persona con acceso para aplicar "merge"(fusionar). Verifica en la pestaña "Files Changed" de la PR para asegurarte de que la prueba automatizada de integración continua (CI) pase. |
There was a problem hiding this comment.
Sugiero traducir todas las instancia de "merge" por fusionar, de la siguiente forma.
| Los "simple fix", como la corrección de un pequeño error tipográfico, pueden "be merged" (fusionarse) directamente por cualquier persona con acceso para aplicar "merge"(fusionar). Verifica en la pestaña "Files Changed" de la PR para asegurarte de que la prueba automatizada de integración continua (CI) pase. | |
| Los arreglos simples, como la corrección de un pequeño error tipográfico, pueden fusionarse directamente por cualquier persona con acceso para fusionar el repositorio. Verifica en la pestaña "Files Changed" de la PR para asegurarte de que la prueba automatizada de integración continua (CI) pase. |
|
|
||
| Los "simple fix", como la corrección de un pequeño error tipográfico, pueden "be merged" (fusionarse) directamente por cualquier persona con acceso para aplicar "merge"(fusionar). Verifica en la pestaña "Files Changed" de la PR para asegurarte de que la prueba automatizada de integración continua (CI) pase. | ||
|
|
||
|  |
There was a problem hiding this comment.
Recuerda que los textos alternativos para las imagenes no se traducen. Estos siempre los encontraras por el siguente patron:
Por favor, devuelvelos a su estado original
|
|
||
|  | ||
|
|
||
|  |
There was a problem hiding this comment.
Devolver a su estado original, ya que es un texto alternativo de una imagen
|
|
||
| ### "Bug fix" (Corrección de error) | ||
|
|
||
| 1. "Bug fixes" deberían ser revisado por el administrador del área relevante, idealmente el mismo que aprobó el "issue" referenciado para su corrección. |
There was a problem hiding this comment.
Falto una "s" en "deberian ser revisados" para mantener la correspondecia en entre el sujeto y el verbo de la oración, que estan en plural
| 1. "Bug fixes" deberían ser revisado por el administrador del área relevante, idealmente el mismo que aprobó el "issue" referenciado para su corrección. | ||
| 2. La pestaña "Files Changed" de la PR se puede utilizar para revisar inicialmente si el "fix" se implementa según lo descrito en la discusión del issue. | ||
| 3. El PR Debería ser probado localmente siempre que sea posible y relevante. El GitHub CLI puede ayudar a agilizar parte del proceso. Ver más abajo en [Consejos y trucos](steward_guidelines.md#tips-tricks). | ||
| - [ ] El "fix"(Corrección) debe abordar suficientemente el problema original. |
There was a problem hiding this comment.
Traducir "fix" por corrección en las siguiente viñetas
| - [ ] El "fix" debe pasar todas las pruebas automatizadas e incluir nuevas pruebas(tests) si son relevantes. | ||
| 4. Si se requieren cambios adicionales, se deben agregar comentarios en línea a las líneas relevantes según se describió anteriormente [aquí](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request). | ||
| - Un bloque de sugerencias también puede ser usado para sugerir cambios específicos:\ | ||
| \ |
There was a problem hiding this comment.
Devolver a su estado original los siguientes tres textos alternativos de imagen
| - Si los comentarios en línea son solo para aclaraciones o discusión, elige "Comment" en lugar de "Request changes":\ | ||
|  | ||
| 5. Una vez que la PR haya sido revisada y no se requieran cambios adicionales, un administrador puede marcar la PR como "Aprobada" eligiendo la opción "Approve" en el paso anterior, con o sin comentarios adicionales. El administrador puede luego solicitar una revisión adicional por otro administrador o encargado de mantenimiento si lo desea, fusionar(merge) la PR si tiene acceso para fusionar(merge access), o solicitar "merge" (fusión) de un encargado de mantenimiento. | ||
| 6. @[all-contributors](https://allcontributors.org/docs/en/emoji-key)El bot debería ser llamado para agregar cualquier nuevo colaborador a la lista de colaboradores en el archivo README.md. Cada tipo de contribución puede ser indicado en lugar de `[contribution` `type]` a continuación, se puede encontrar la lista completa de tipos de contribuciones disponibles en el enlace anterior. |
There was a problem hiding this comment.
| 6. @[all-contributors](https://allcontributors.org/docs/en/emoji-key)El bot debería ser llamado para agregar cualquier nuevo colaborador a la lista de colaboradores en el archivo README.md. Cada tipo de contribución puede ser indicado en lugar de `[contribution` `type]` a continuación, se puede encontrar la lista completa de tipos de contribuciones disponibles en el enlace anterior. | |
| 6. El bot @[all-contributors](https://allcontributors.org/docs/en/emoji-key) debería ser llamado para agregar cualquier nuevo colaborador a la lista de colaboradores en el archivo README.md. Cada tipo de contribución puede ser indicado en lugar de `[contribution` `type]` a continuación, se puede encontrar la lista completa de tipos de contribuciones disponibles en el enlace anterior. |
|
|
||
| Las PRs de Dependabot generalmente solo son visibles para los administradores del repositorio, así que si esto no aplica a ti, por favor omite esta sección. | ||
|
|
||
| - Las PRs de Dependabot pueden fusionarse directamente si la actualización de la versión es una [semver](https://semver.org/) versión de parche y la prueba automatizada de CI ha pasado. |
There was a problem hiding this comment.
| - Las PRs de Dependabot pueden fusionarse directamente si la actualización de la versión es una [semver](https://semver.org/) versión de parche y la prueba automatizada de CI ha pasado. | |
| - Las PRs de Dependabot pueden fusionarse directamente si la actualización de la versión es una versión de parche [semver](https://semver.org/) y la prueba automatizada de CI ha pasado. |
|
Muchas gracias, Marce! Resumire mis comentarios:
El resto lo encontraras más arriba. |
| # Directrices para Administradores | ||
|
|
||
|
|
||
| Ya sea que recién te hayas unido a nosotros como administrador, seas un mantenedor experimentado de p5.js, o estés en algún punto intermedio, esta guía contiene información, así como consejos y trucos que te ayudarán a contribuir de manera efectiva a p5.js. La mayor parte de lo que se escribe aquí son pautas, a menos que se indique lo contrario, lo que significa que puedes adaptar las prácticas mostradas aquí para que se ajusten a tu flujo de trabajo. |
| 1. Como parte del compromiso de p5.js de aumentar el acceso, una solicitud de función(feature request) debe justificar cómo aumenta el acceso de p5.js a comunidades que históricamente han sido marginadas en el campo. Más detalles están disponibles aquí[here](access.md). | ||
| - If a feature request does not have the "Increasing Access" field sufficiently filled out, you can ask the issue author how the feature increases access. | ||
| - Si una solicitud de función(feature request) no tiene suficientemente completado el campo "Aumento de Acceso"("Increasing Access"), puedes preguntar al autor del problema(issue autor) cómo la función(feature) aumenta el acceso. | ||
| - La declaración de acceso de una función puede ser proporcionada por un miembro diferente de la comunidad, incluidos los revisores de problemas(issue reviewers). |
There was a problem hiding this comment.
sigue aun el problema de la falta de espacio antes de abirr parentesis
CC: @Guirdo
| - Por ejemplo, una solicitud para agregar una nueva forma primitiva de dibujo puede ser considerada, pero una solicitud para adoptar un protocolo de Internet de las cosas basado en el navegador (browser-based IOT protocol) probablemente estará fuera de alcance. | ||
| - En general, el alcance de p5.js debería ser relativamente estrecho para evitar un exceso de características poco utilizadas. | ||
| - Si una función no encaja en el alcance de p5.js, sugiere al autor del problema(issue) que implemente la función(feature) como una biblioteca complementaria(addon library). | ||
| - Si no está claro si encaja o no, puede ser una buena idea sugerir hacer una biblioteca complementaria(addon library)como una prueba de concepto(proof-of-concept).Esto ayuda a dar a los usuarios una forma de usar la función(feature), proporciona un ejemplo mucho más concreto de su uso e importancia, y no necesariamente necesita ser una solución tan completa como una función completamente integrada. Puede integrarse en el núcleo(the core) de p5.js más adelante si corresponde. |
| - ¿Es probable que la función(feature) propuesta cause un cambio incompatible? | ||
| - ¿Entrará en conflicto con las funciones y variables existentes de p5.js? | ||
| - ¿Entrará en conflicto con los bocetos típicos(typical sketches) ya escritos para p5.js? | ||
| - Las funciones que probablemente causen conflictos, como las mencionadas anteriormente, se consideran cambios incompatibles. Sin un lanzamiento de [Lanzamiento de versión mayor](https://docs.npmjs.com/about-semantic-versioning), no deberíamos realizar cambios incompatibles en p5.js. |
There was a problem hiding this comment.
falta espacio entre "un" y "Lanzamiento"
| 3. Si el requisito de acceso y otras consideraciones han sido cumplidas, al menos dos administradores o mantenedores deben aprobar la nueva solicitud de función antes de que comience el trabajo hacia una PR. El proceso de revisión de PR para nuevas funciones está documentado a continuación. | ||
|
|
||
|
|
||
| ### "Feature enhancement" (Mejora de funciones) |
There was a problem hiding this comment.
como se mencionó varias veces utilizaremos Funcionalidad en lugar de funcion
CC: @Guirdo
Adding _sketches_
|
hola Marce, por favor actualiza el nombre del PR cómo se comentó en el grupo, estoy revisando tus cambios |
Replacing PR by _pull request_
| 1. "Bug fixes"(Corrección de errores) deberían ser revisado por el administrador del área relevante, idealmente el mismo que aprobó el _issue_ referenciado para su corrección. | ||
| 2. La pestaña "Files Changed" de la _pull request_ se puede utilizar para revisar inicialmente si el _fix_ (la ccorrección) se implementa según lo descrito en la discusión del _issue_. | ||
| 3. La _pull request_ Debería ser probada localmente siempre que sea posible y relevante. El GitHub CLI puede ayudar a agilizar parte del proceso. Ver más abajo en [Consejos y trucos](steward_guidelines.md#tips-tricks). | ||
| - [ ] "La Corrección" debe abordar suficientemente el _issue_ original. |
There was a problem hiding this comment.
Las comillas en "La corrección" no son necesarias
There was a problem hiding this comment.
Gracias, se me fue. Comillas borradas.
|
Reemplazada Cris. |
| - [Tarea Principal de Construcción](steward_guidelines.md#tarea-principal-de-construcción) | ||
| - [Tarea Variada](steward_guidelines.md#tarea-variada) | ||
| - [Proceso de Lanzamiento](steward_guidelines.md#proceso-de-lanzamiento) | ||
| - [Consejos y Trucos](steward_guidelines.md#tips--consejos-y-trucos) |
There was a problem hiding this comment.
Aquí se te paso quitar "tips--" en el enlace
| - [Consejos y Trucos](steward_guidelines.md#tips--consejos-y-trucos) | |
| - [Consejos y Trucos](steward_guidelines.md#tips--consejos-y-trucos) |
There was a problem hiding this comment.
Gracias, "tips-- borrado"
| @@ -0,0 +1,381 @@ | |||
| # Directrices para Administradores | |||
There was a problem hiding this comment.
Traducir "stewards" como "supervisores"
There was a problem hiding this comment.
Remplazado, excepto por el admin mencionado tal cual en dependabot:
Dependabot PRs are usually only visible to "repo admins " so if this does not apply to you, please skip this section.
There was a problem hiding this comment.
Muchas gracias por los arreglos Marce. Tengo dos comentarios importantes:
- Corregir el
pathrelativo de las imágenes para poder ver el render como dijo Sebastiona, agregando el "../' antes del path que está ahorita - Veo todavía inconnsistencias entre el espacio y los paréntesis por ejemplo aun tienes fragmentos como el que sigue:
-
Texto(contenido). cuando falta un espacio asi:
-
Text (contenido). El problema es que no es consistente en tu mismo texto, si todos fueran con espacio o sin espacio seria menos problema, por favor agrega un espacio en los que falta.
-
Con estos dos detalles ya estoy listo para aprobar tu PR. Muchas gracias
CC: @Guirdo
| - [ ] El fix debe pasar todas las pruebas automatizadas e incluir nuevas pruebas(tests) si son relevantes. | ||
| 4. Si se requieren cambios adicionales, se deben agregar comentarios en línea a las líneas relevantes según se describió anteriormente [here](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request). | ||
| - Un bloque de sugerencias también puede ser usado para sugerir cambios específicos:\ | ||
| \ |
There was a problem hiding this comment.
De acuerdo con sebastian, por favor actualiza el path a como dice él con "../" para todas, ya que no se están dibujando
| - Intente solucionarlo usted mismo o agregue la etiqueta `help wanted` para señalar que es un _issue_ que necesita solución. | ||
| 3. Si el error no se puede replicar: | ||
| - Solicite información adicional si aún no se ha proporcionado en la plantilla (versión de p5.js, versión del navegador, versión del sistema operativo, etc). | ||
| - Si su entorno de prueba difiere de lo que se informa en el _issue_(por ejemplo,un navegador o sistema operativo diferente): |
There was a problem hiding this comment.
aqui falta un espacio antes del parentesis
There was a problem hiding this comment.
Gracias, corregido.
There was a problem hiding this comment.
también se agrego ../ a las imagenes.
| - Si su entorno de prueba difiere de lo que se informa en el _issue_(por ejemplo,un navegador o sistema operativo diferente): | ||
| - Deje un comentario diciendo que no puede replicar en su entorno específico. | ||
| - Agregue una etiqueta `help wanted` al _issue_ incidente y pida a alguien más con la configuración especificada en el _issue_ que intente replicar el error. | ||
| - A veces, los "bugs"(errores) solo ocurren al usar el editor web y no al probar localmente. En este caso, el _issue_ debería ser redirigido al [repositorio del editor web](https://github.com/processing/p5.js-web-editor). |
There was a problem hiding this comment.
issue (por ejemplo,un navegador o sistema operativo diferente):
- A veces, los "bugs" (errores)
espacio aplicado antes de parentesis.
|
|
||
| Este tipo de _issue_ tiene una plantilla mínima de discusión y debería ser utilizada para recopilar comentarios y retroalimentaciones sobre un tema en general antes de consolidarlo en algo más específico, como una solicitud de función. Estos _issues_ de discusión pueden cerrarse cuando finaliza la conversación y se han creado los _issues_ más específicos resultantes: | ||
|
|
||
| - Si se abre un _issue_ como una discusión pero debería ser, por ejemplo, un _bug report_(informe de error), se debe aplicar la etiqueta correcta y quitar la etiqueta de "discusión". Además, se debe solicitar información adicional sobre el error al autor si aún no se ha incluido. |
There was a problem hiding this comment.
un bug report (informe de error), espacio aplicado.
|
|
||
| ### Corrección de Error | ||
|
|
||
| 1. "Bug fixes"(Corrección de errores) deberían ser revisado por el supervisor del área relevante, idealmente el mismo que aprobó el _issue_ referenciado para su corrección. |
| - Si se requieren múltiples cambios, no agregues comentarios de una sola línea muchas veces. En su lugar, sigue el procedimiento documentado [aquí](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request) para hacer comentarios de varias líneas y una sola solicitud de cambios (change request). | ||
| - Si los comentarios en línea son solo para aclaraciones o discusión, elige "Comment" en lugar de "Request changes":\ | ||
|  | ||
| 5. Una vez que la _pull request_ haya sido revisada y no se requieran cambios adicionales, un supervisor puede marcar la _pull request_ como "Aprobada" eligiendo la opción "Approve" en el paso anterior, con o sin comentarios adicionales. El supervisor puede luego solicitar una revisión adicional por otro supervisor o responsable de mantenimiento si lo desea, fusionar(merge) la _pull request_ si tiene acceso para fusionar(merge access), o solicitar "merge" (fusión) de un responsable de mantenimiento. |
There was a problem hiding this comment.
fusionar (merge access), o solicitar "merge" (fusión) , espacio agregado.
|
|
||
| ### Nuevas funcionalidades/Mejora de Funcionalidades | ||
|
|
||
| El proceso para una _pull request_ de _new feature_ (nuevas funcionalidades),o _feature_enhacement_(mejora de funcionalidades) es similar a las correcciones de errores,pero solo con una diferencia notable: |
There was a problem hiding this comment.
El proceso para una pull request de new feature (nuevas funcionalidades),o feature_enhacement (mejora de funcionalidades) espacio aplicado.
|
@ml008008 falta también actualizar el nombre del PR al formato correcto por favor |
ml008008
changed the title
|
@Qianqianye @limzykenneth This PR is also ready |
|
Looks great, thanks @ml008008 @holomorfo @Guirdo 💗 |
|
I've put up a pull request to add @ml008008! 🎉 |
Resolves #[Add issue number here]
Changes:
Screenshots of the change:
PR Checklist
npm run lintpasses