Skip to content

Traducción c-api/refcounting.po #3386

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 12 commits into from
May 14, 2025
Merged

Traducción c-api/refcounting.po #3386

merged 12 commits into from
May 14, 2025

Conversation

M-E-K-J-2102
Copy link
Contributor

Closes #3260

Copy link

github-actions bot commented Apr 15, 2025

Entries missing translation, details follow:

Reason File Line
untranslated c-api/refcounting.po 124
untranslated c-api/refcounting.po 170
untranslated c-api/refcounting.po 180
untranslated c-api/refcounting.po 230
untranslated c-api/refcounting.po 333
untranslated c-api/refcounting.po 343

Copy link
Collaborator

@mmmarcos mmmarcos left a comment

Choose a reason for hiding this comment

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

Gracias por esta contribución! Ahí te dejo algunos comentarios.

Recuerda revisar las entradas marcadas como "fuzzy" en el archivo. Después de traducir, o de corregir la traducción existente, debés borrar la línea que dice

#, fuzzy

Comment on lines 262 to 263
"método :meth:`__del__`). Si las excepciones en dicho código no se "
"propagan, el código ejecutado tendrá acceso libre a todas las variables "
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"método :meth:`__del__`). Si las excepciones en dicho código no se "
"propagan, el código ejecutado tendrá acceso libre a todas las variables "
"método :meth:`__del__`). Si las excepciones en dicho código no se "
"método :meth:`~object.__del__`). Mientras las excepciones en dicho código no sean "
"propagadas, el código ejecutado tendrá acceso libre a todas las variables "

@M-E-K-J-2102 M-E-K-J-2102 requested a review from mmmarcos April 19, 2025 16:27
@M-E-K-J-2102
Copy link
Contributor Author

Hola, ¿hay algo más que tenga que actualizar antes de que me aprueben mis contribuciones? ¡Gracias!

Copy link
Collaborator

@mmmarcos mmmarcos left a comment

Choose a reason for hiding this comment

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

Gracias por corregir los fuzzy! Ahí van unas ultimas sugerencias

M-E-K-J-2102 and others added 7 commits April 23, 2025 13:13
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Co-authored-by: Marcos Medrano <786907+mmmarcos@users.noreply.github.com>
Copy link
Collaborator

@mmmarcos mmmarcos left a comment

Choose a reason for hiding this comment

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

@M-E-K-J-2102 ahora sí me parece todo bien, gracias por la contribución! 👍
(y perdón por la demora 🙈)

@mmmarcos mmmarcos merged commit cacb339 into python:3.13 May 14, 2025
3 checks passed
@rtobar
Copy link
Collaborator

rtobar commented May 16, 2025

¡Muchas gracias @M-E-K-J-2102 por tu primera traducción!

Lamentablemente esta contribución fue mergeada incluyendo algunos algunos errores. La interfaz de GitHub cambió hace poco, y probablemente no era obvio que el trabajo principal de CI que hace todos los chequeos necesitaba ser aprovado (dado que era la primera contribución de @M-E-K-J-2102) antes de que se ejecutara. El comentario al comienzo del PR también aún muestra entradas sin traducir o aún en estado fuzzy.

@M-E-K-J-2102 ¿te animarías a intentarlo una vez más? Tendrías que crear un nuevo branch y comenar a trabajar desde ahí, traduciendo las entradas que falten, y corrigiendo los errores que arroje CI. Ahora que ya hiciste una primera contribución CI va a ejecutarse en su totalidad sin necesidad de que nadie lo apruebe, así que los errores van a ser más obvios.

@mmmarcos
Copy link
Collaborator

Lamentablemente esta contribución fue mergeada incluyendo algunos algunos errores. La interfaz de GitHub cambió hace poco, y probablemente no era obvio que el trabajo principal de CI que hace todos los chequeos necesitaba ser aprovado (dado que era la primera contribución de @M-E-K-J-2102) antes de que se ejecutara. El comentario al comienzo del PR también aún muestra entradas sin traducir o aún en estado fuzzy.

@rtobar Ouch, gracias por la info! no había visto que la CI esperaba una aprobación para ejecutarse.

@M-E-K-J-2102 ¿te animarías a intentarlo una vez más? Tendrías que crear un nuevo branch y comenar a trabajar desde ahí, traduciendo las entradas que falten, y corrigiendo los errores que arroje CI. Ahora que ya hiciste una primera contribución CI va a ejecutarse en su totalidad sin necesidad de que nadie lo apruebe, así que los errores van a ser más obvios.

@rtobar @M-E-K-J-2102 si les parece yo puedo ocuparme, por las molestias ocasionadas ^^

@rtobar
Copy link
Collaborator

rtobar commented May 16, 2025

Por mi lado, ningún problema! Y sin dramas, no hay molestias ocasionadas 😉 👍

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/refcounting.po
3 participants