-
Notifications
You must be signed in to change notification settings - Fork 396
Traducido archivo library/hashlib #2301
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
Conversation
Listo para revisión, se revisaron los fuzzy además. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fmoradev una vez más gracias por el súper trabajo :)
Dejo un par de comentarios, creo que va a haber que pasar un powrap después de aplicarlos.
…on-docs-es into traduccion-library-hashlib
…on-docs-es into traduccion-library-hashlib
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perdón @fmoradev por la pausa, te dejo un par de comentarios más que se me pasaron la primera vez. Voy a probar también a ver qué pasa con lo del link a stackexchange, a ver si logro dar con una respuesta satisfactoria.
@rtobar podrías ayudarme a identificar por qué no pasa el powrap? ya lo corrí dos veces y no sé por qué esta fallando el test por favor. |
En qué sistema operativo estás? y qué versión de |
@cmaureir te edité el comentario por accidente :(:(:(:( |
Como intuye @cmaureir quizás estás trabajando desde Windows, o tienes una versión antigua de gettext (en GitHub Actions corremos con 0.21.4 ahora, así que supongo que la restricción es que sea al menos 0.21.1). De todas formas, acabo de hacer un push con el último fix a powrap y con esto ya hacemos el merge. En el futuro quizás sería útil ver dónde powrap cree que hay problemas, voy a abrir un PR separado oar eso. |
Estoy trabajando en Debian 11, con gettext 0.21.4 |
Closes #1951