Traducido archivo library/dataclasses#744
Conversation
Corrección de errores de Travis por inconsistencias de términos
Corrección de errores gramaticales y otros cambios menores.
Estandarizado "devolver" por retornar y algunos cambios menores.
narvmtz
left a comment
There was a problem hiding this comment.
¡Gracias por la traducción @fjsevilla-dev!
Dejo un par de sugerencias de forma, espero sirvan para acercarnos al merge del archivo :)
Voy en revisión hasta la línea 400 si alguien más se anima a seguirlo revisando, de lo contrario lo retomo luego.
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
|
¡Gracias por la revisión @narvmtz! He aceptado todas las sugerencias menos una de momento, arriba comento mi duda. El build falla por powrap, que no está nada contento con que se cambien las longitudes de linea sin preguntarle XD. En principio lo voy a dejar así, cuando la revisión termine aplico un commit final para corregir esto y agún otro fallo que pudiera surgir. |
|
A mi el documento me parece bien, igual esperaría por la revisión de @narvmtz en las siguientes líneas a ver si ella encuentra alguna cosita. |
narvmtz
left a comment
There was a problem hiding this comment.
Dejo un par de sugerencias más y creo que luego de resolverlas, pasarle powrap y que pase todos los checks estaríamos listos para el merge :)
Gran trabajo con el archivo @fjsevilla-dev y con la revisión @cmaureir 🎉, muchas gracias a ambos.
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
narvmtz
left a comment
There was a problem hiding this comment.
@fjsevilla-dev solo faltaría pasarle powrap al archivo y en mi opinión quedaría listo :)
|
Listo. He aplicado powrap y corregido un par de errores, un punto y seguido que faltaba y una comilla invertida que había quedado por ahí solitaria. Tras leer el build de la documentación creo que ha quedado bien y listo para el merge. Gracias a ambos por las sugerencias 👍 . |
|
Muchas gracias @fjsevilla-dev por la traducción 🎉 🎉 |
Closes #462