Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Mar 6, 2023

Closes #1893

@ghost ghost marked this pull request as ready for review March 7, 2023 13:28
@ghost
Copy link
Author

ghost commented Mar 7, 2023

El powrap me sigue fallando por alguna razón, podrían ayudarme?

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.

@fmoradev Ahí corrí un powrap y subí los cambios, ojalá ya ahora pasen los tests.

Te dejo un comentario, y una duda: ¿qué pasa con las entradas fuzzy? El issue dice que hay dos, pero en el PR no hay ninguna entrada fuzzy que esté siendo trabajadada.

Co-authored-by: rtobar <rtobarc@gmail.com>
@ghost
Copy link
Author

ghost commented Mar 7, 2023

@fmoradev Ahí corrí un powrap y subí los cambios, ojalá ya ahora pasen los tests.

Te dejo un comentario, y una duda: ¿qué pasa con las entradas fuzzy? El issue dice que hay dos, pero en el PR no hay ninguna entrada fuzzy que esté siendo trabajadada.

Hay dos en el archivo como indicas @rtobar ni idea por que no aparece en el PR.

@cmaureir
Copy link
Collaborator

cmaureir commented Mar 7, 2023

@fmoradev Ahí corrí un powrap y subí los cambios, ojalá ya ahora pasen los tests.
Te dejo un comentario, y una duda: ¿qué pasa con las entradas fuzzy? El issue dice que hay dos, pero en el PR no hay ninguna entrada fuzzy que esté siendo trabajadada.

Hay dos en el archivo como indicas @rtobar ni idea por que no aparece en el PR.

creo que el tema va en por qué no están actualizadas/modificadas. Si tienes seguridad que la traducción que tienen está correcta, tendrías que quitarles el identificador #, fuzzy.

@ghost
Copy link
Author

ghost commented Mar 7, 2023

Fuzzy revisados @rtobar @cmaureir

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.

Gracia @fmoradev por fijarte en las entradas que estaban fuzzy. Te dejo un par de comentarios más y estamos.

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.

@fmoradev ahora sí estamos listos, me tomé la libertad de coregir un último detalle que faltaba arreglar. En cuanto pasen los tests le doy al merge :)

@ghost
Copy link
Author

ghost commented Mar 9, 2023

@fmoradev ahora sí estamos listos, me tomé la libertad de coregir un último detalle que faltaba arreglar. En cuanto pasen los tests le doy al merge :)

Genial. @rtobar

@cmaureir cmaureir changed the title traducido-archivo-howto-urllib2.po Traducido archivo howto/urllib2 Mar 9, 2023
@rtobar rtobar merged commit 09d589e into python:3.11 Mar 9, 2023
@ghost ghost deleted the traduccion-howto-urllib2 branch March 9, 2023 13:48
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 'howto/urllib2.po'
3 participants