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

Traducido archivo c-api/type.po #2370

Merged
merged 14 commits into from
Aug 7, 2023
Merged

Traducido archivo c-api/type.po #2370

merged 14 commits into from
Aug 7, 2023

Conversation

igmtz
Copy link
Contributor

@igmtz igmtz commented Apr 8, 2023

Closes #2053

@igmtz igmtz mentioned this pull request Apr 8, 2023
@rtobar
Copy link
Collaborator

rtobar commented Apr 11, 2023

@igmtz muchas gracias por tu primera contribución! Le eché una mirada rápida y creo que en general se ve bien, pero más rato le doy una revisión más minuciosa. ¡Mucho ánimo! 💪

Respecto al powrap: en https://python-docs-es.readthedocs.io/es/3.11/CONTRIBUTING.html hay varios tips respecto a cómo usar powrap. Recuerda que después de correrlo tienes que hacer un commit con los cambios, ésto no ocurre automáticamente (¿quizás eso es lo que te hace falta?).

Otra cosa: veo que hay traducciones para las entradas a las que le fataba una, pero en #2053 dice que además hay 4 entradas "fuzzy"; es decir, hace falta revisar sus traducciones para revisar que estén correctas (normalmente hace falta ajustar algo porque el contenido original cambió desde la versión anterior de python). En poedit estas entradas aparecen marcadas en amarillo, o dicen "Needs work". ¿Le podrías echar una mirada a esas? La idea es tener el archivo traducido al 100%.

@igmtz
Copy link
Contributor Author

igmtz commented Apr 11, 2023

@rtobar muchas gracias! 😁 Sí! Justo he tenido varios problemas con powrap porque después de instalarlo no reconoce el comando, trato de seguir las instrucciones de la guía de contribución pero parece que sigo haciendo algo mal :c

Seguiré intentando y checo las entrada "fuzzy" para que esté completo, muchas gracias por la observación. En un rato subo otro commit

Copy link
Contributor

@aandrea aandrea left a comment

Choose a reason for hiding this comment

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

Hola, me parece que está muy bien en general. :)
Dejo algunas sugerencias de mejoras y una corrección de un error en una referencia.

c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
c-api/type.po Outdated Show resolved Hide resolved
@cmaureir
Copy link
Collaborator

cmaureir commented Aug 7, 2023

Si pasa, hacemos merge

@cmaureir
Copy link
Collaborator

cmaureir commented Aug 7, 2023

mmm el error viene de otro lado, con lo que algo pasó con el CI

whatsnew/3.11.po:476:annotating
whatsnew/3.11.po:476:instance
whatsnew/3.11.po:4041:Tier
whatsnew/3.11.po:4041:tier
whatsnew/3.11.po:5277:migration
whatsnew/3.11.po:5277:guidance
whatsnew/3.11.po:5277:alternative

ese archivo se modificó hace tiempo. A ver si ahora pasa.

@cmaureir cmaureir merged commit c6aff48 into python:3.11 Aug 7, 2023
1 check passed
@cmaureir
Copy link
Collaborator

cmaureir commented Aug 7, 2023

gracias @igmtz por la traducción y @aandrea por la revisión 🎉

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 'c-api/type.po'
4 participants