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

Traduccion library/crypto #1195

Merged
merged 4 commits into from Jan 17, 2021
Merged

Conversation

@framorac
Copy link
Contributor

@framorac framorac commented Jan 8, 2021

Favor revisar.
Closes #907

@cmaureir cmaureir changed the title Traduccion library crypto Traduccion library/crypto Jan 10, 2021
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jan 10, 2021

Gracias por la traducción @framorac .
Lo único que falta es que le pases powrap a tu archivo: https://python-docs-es.readthedocs.io/es/3.8/reviewers-guide.html?highlight=powrap#powrap-falla

@framorac
Copy link
Contributor Author

@framorac framorac commented Jan 11, 2021

Ya pasé powrap a mi archivo y subí a mi rama, es necesario un nuevo PR?

@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jan 12, 2021

Ya pasé powrap a mi archivo y subí a mi rama, es necesario un nuevo PR?

no, solo eso para que pase el build y acá mismo en el mismo PR es como trabajamos 👍

@framorac
Copy link
Contributor Author

@framorac framorac commented Jan 12, 2021

Ya pasé powrap a mi archivo y subí a mi rama, es necesario un nuevo PR?

no, solo eso para que pase el build y acá mismo en el mismo PR es como trabajamos +1

Perfecto, aunque falló el build de docs, no sé si es por que primero se debe aprobar el PR. En caso contrario, no sé como corregir por mi parte ese fallo. Saludos @cmaureir

@cmaureir cmaureir merged commit b68f4e4 into python:3.8 Jan 17, 2021
2 checks passed
2 checks passed
Test
Details
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jan 17, 2021

Muchas gracias @framorac 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.