Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Traducción library/sqlite3.po #2237

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

Merged
merged 122 commits into from
Jan 18, 2023
Merged

Conversation

Alfareiza
Copy link
Contributor

@Alfareiza Alfareiza commented Nov 30, 2022

Closes #1952

@Alfareiza
Copy link
Contributor Author

@cmaureir Algunos errores al ejecutar python scripts/check_spell.py aparecen en el github actions, sin embargo en mi pc no surgieron.

@rtobar
Copy link
Collaborator

rtobar commented Dec 26, 2022

@cmaureir Algunos errores al ejecutar python scripts/check_spell.py aparecen en el github actions, sin embargo en mi pc no surgieron.

El error relevante, y que parece genuino, es:

library/sqlite3.po:1412:personzliada

@Alfareiza
Copy link
Contributor Author

Terminado!

@rtobar rtobar marked this pull request as ready for review December 28, 2022 06:39
@rtobar
Copy link
Collaborator

rtobar commented Dec 28, 2022

@Alfareiza Tremendo trabajo!!

Va a tomar un poco de tiempo revisar la traducción, pero por el momento un par de cosas:

  • Edité la descripción para que el issue correspondiente se cierre una vez que esté PR sea mergeado.
  • Marqué el PR como listo para ser revisado.
  • Hay un error en los tests. El problema parece ser que hace falta correr powrap sobre el archivo. Hay una guía al respecto en la documentación de la traducción en caso de que no estés familiarizada/o con el tema.

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Revisado hasta (nueva) línea 504

@Alfareiza
Copy link
Contributor Author

Alfareiza commented Dec 28, 2022

@rtobar, gracias por tu tiempo para revisar también, y sí, fue tremendo trabajo de algo más de un mes, creo. A continuación mi respuesta luego de tu revisión.

Cronológicamente hice lo siguiente:

  • Apliqué algunas correciones comitando tus sugerencias, aqui mismo por github.
  • En este commit realicé lo siguiente:
    • Apliqué powrap así:
      $ powrap library/sqlite3po
    • Luego apliqué algunas de tus sugerencias desde mi branch local.
  • Luego me di cuenta que salió error de powrap y le apliqué de nuevo al archivo el mismo comando arriba mencionado, através de este último commit

@rtobar
Copy link
Collaborator

rtobar commented Dec 28, 2022

@Alfareiza acabo de ir por la lista de correcciones y se ve que va mejorando la cosa :-). Si te fijas en el detalle del error de los tests ahora ya no hay problema con powrap sino que con la ortografía, está reclamando que "fetchone" está mal escrito. Si la palabra aparece dentro de la prosa entonces hace falta agregarla al diccionario, pero hay que fijarse también que no corresponda a un texto que falte por traducir.

Vi que atendiste los comentarios que te dejé, los primos y los últimos, pero ninguno de los que están entremedio. La interfaz de GitHub no muestra todos los comentarios subes que son muchos, y hay que pedirle que expanda la lista. ¿Fue ésto lo que pasó, o estás trabajando en ellos?

Perdón por algunos comentarios repetidos también, la interfaz de GH se estaba mareando y no me actualizaba bien la página después de poner algunos comentarios y no sabía si habían quedado subidos o no, por eso es que aparecen más de una vez.

Alfareiza and others added 7 commits December 29, 2022 14:31
Colocando palabra conteniendo en vez de contiene, y agregando al final de la frase "de esa fila", para mejor contextualización.

Co-authored-by: rtobar <[email protected]>
Cambiando "más elementos" por "filas más"

Co-authored-by: rtobar <[email protected]>
Cambiando "la" por "una"

Co-authored-by: rtobar <[email protected]>
Cambiando "lecturas de intereś" para "para lecturas de interés"

Co-authored-by: rtobar <[email protected]>
Cambiando "Abrir" por "Abre"

Co-authored-by: rtobar <[email protected]>
@Alfareiza
Copy link
Contributor Author

Esos errores están resultando en el archivo :1:, te refieres a esos mismos?, de ser así, voy a necesitar orientaciones sobre como abordarlos

Alfareiza and others added 12 commits January 14, 2023 14:23
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Collaborator

Muchas gracias @Alfareiza

@cmaureir cmaureir requested a review from rtobar January 16, 2023 12:40
@rtobar rtobar merged commit afe94af into python:3.11 Jan 18, 2023
@rtobar
Copy link
Collaborator

rtobar commented Jan 18, 2023

Muchas gracias @Alfareiza por el tremendo trabajo y dedicación, y mil disculpas por no haber podido seguir revisando con la atención con la que empecé -- se terminaron las vacaciones y el tiempo disponible cayó al mínimo! @cmaureir le dio el empuje final que faltaba, así que confío que quedó todo súper :)

@Alfareiza
Copy link
Contributor Author

Muchas gracias @Alfareiza por el tremendo trabajo y dedicación, y mil disculpas por no haber podido seguir revisando con la atención con la que empecé -- se terminaron las vacaciones y el tiempo disponible cayó al mínimo! @cmaureir le dio el empuje final que faltaba, así que confío que quedó todo súper :)

Gracias a ti, sin duda, tu aporte también hizo parte fundamental en conseguir la traducción completa.

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

Successfully merging this pull request may close these issues.

Translate 'library/sqlite3.po'
3 participants