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

Jump to bottom

Traducido archivo howto/urllib2 #2340

Merged
merged 10 commits into from Mar 9, 2023
Merged

Traducido archivo howto/urllib2 #2340

merged 10 commits into from Mar 9, 2023

Conversation

fmoradev
Copy link
Contributor

@fmoradev fmoradev commented Mar 6, 2023

Closes #1893

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

fmoradev commented Mar 7, 2023

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

rtobar
rtobar reviewed Mar 7, 2023
View reviewed changes
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.

howto/urllib2.po Outdated Show resolved Hide resolved
@fmoradev
Copy link
Contributor Author

fmoradev 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.

@fmoradev
Copy link
Contributor Author

fmoradev commented Mar 7, 2023

Fuzzy revisados @rtobar @cmaureir

rtobar
rtobar reviewed Mar 8, 2023
View reviewed changes
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.

howto/urllib2.po Show resolved Hide resolved
howto/urllib2.po Show resolved Hide resolved
howto/urllib2.po Show resolved Hide resolved
rtobar
rtobar reviewed Mar 9, 2023
View reviewed changes
howto/urllib2.po Outdated Show resolved Hide resolved
rtobar
rtobar approved these changes Mar 9, 2023
View reviewed changes
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 :)

@fmoradev
Copy link
Contributor Author

fmoradev 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
2 checks passed
@fmoradev fmoradev 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
Reviewers

@rtobar rtobar rtobar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Translate 'howto/urllib2.po'
3 participants
@fmoradev @cmaureir @rtobar