Skip to content

Traducido library/cmath.po #624

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 13 commits into from
Sep 26, 2020
Merged

Traducido library/cmath.po #624

merged 13 commits into from
Sep 26, 2020

Conversation

Buzzeitor30
Copy link
Contributor

Closes #520
Se recomienda revisar traducción ( es la primera vez que hago algo así).

@cmaureir
Copy link
Collaborator

Me picaban los dedos por revisar este archivo :) Gracias @Buzzeitor30
Te recomiendo que mires los detalles de travis, y te darás cuenta que hay muchas palabras que hay que mirar: puede ser que estén mal escritas, o que sean palabras en inglés, etc.

Si tienes alguna duda con alguna de ellas, avisame!

@Buzzeitor30
Copy link
Contributor Author

Perdona que pregunte, ¿pero que son los detalles de travis?

@cmaureir
Copy link
Collaborator

Screenshot_2020-07-26 Traducido archivo cmath po by Buzzeitor30 · Pull Request #624 · python python-docs-es

Si miras esa imagen, al lado derecho de "continuous-intergration-....." aparece un link que dice "Details"
así puedes mirar los problemas del build.

@Buzzeitor30
Copy link
Contributor Author

Buenas un par de dudas, me detecta travis que la palabra "arcocoseno" no es correcta y, además al final de esta página de la documentación se adjunta un libro que trata el tema de los números complejos pero su titulo esta en inglés y es detectado como error. ¿Que hago respecto a esto?
Gracias y un saludo, cuando me respondas a esto haré la pull request y espero que esta vez lo haya hecho bien

@cmaureir
Copy link
Collaborator

Buenas un par de dudas, me detecta travis que la palabra "arcocoseno" no es correcta y, además al final de esta página de la documentación se adjunta un libro que trata el tema de los números complejos pero su titulo esta en inglés y es detectado como error. ¿Que hago respecto a esto?
Gracias y un saludo, cuando me respondas a esto haré la pull request y espero que esta vez lo haya hecho bien

Para la palabra, la agregas a tu diccionario https://python-docs-es.readthedocs.io/es/3.8/faq.html#como-agrego-una-palabra-al-diccionario
Y para el libro, encierra el comienzo y final con asteriscos y será omitido. Te lo dejo como cambio en el mismo código.

@cmaureir
Copy link
Collaborator

@Buzzeitor30 te puedo ayudar en algo para continuar con este PR?

@Buzzeitor30
Copy link
Contributor Author

@Buzzeitor30 te puedo ayudar en algo para continuar con este PR?

Estoy de vacaciones al volver lo hago

@Buzzeitor30
Copy link
Contributor Author

No consigo agregar la palabra que quiero a la carpeta dictionaries, aún así creo que está bien el resto, voy a hacer pull request y a ver si lo puedes solucionar tu de algun forma

@Buzzeitor30
Copy link
Contributor Author

No consigo agregar la palabra que quiero a la carpeta dictionaries, aún así creo que está bien el resto, voy a hacer pull request y a ver si lo puedes solucionar tu de algun forma

Vale voy bastante perdido agradeceria algo de ayuda por Discord si es posible dejo aquí mi usuario:
Buzzeitor#7107

@cmaureir
Copy link
Collaborator

cmaureir commented Aug 17, 2020

No consigo agregar la palabra que quiero a la carpeta dictionaries, aún así creo que está bien el resto, voy a hacer pull request y a ver si lo puedes solucionar tu de algun forma

Hola @Buzzeitor30, el grupo se reune en telegram, discord fue usado solo para los sprints de la EuroPython.
Si no tienes el directorio dictionaries/ significa que tu rama no está actualizada, si ese es el caso, solo debes hacer en tu rama local:

git fetch --all
git merge upstream/3.8 # quizás esto abre tu editor para hacer un commit

(más información de como mantener la rama local actualizada https://python-docs-es.readthedocs.io/es/3.8/faq.html#mantener-actualizada )

Para agregar la palabra, solo creas un archivo dentro de ese directorio y agregas una palabra por línea, por ejemplo si quiero agregar la palabara 'ordenador' y 'mouse':

$ TuEditor dictionaries/library_math.txt

y dentro dejas:

ordenador
mouse

Si miras el resultado del último build, hay varias palabras que están mal escritas y por eso no son reconocida. Te recomiendo que vayas buscandolas y arreglandolas y luego de las que te queden, las agregas al diccionario.

@Buzzeitor30
Copy link
Contributor Author

Ya he hecho lo que me dices, y he creado el archivo dentro de la carpeta dictionaries pero antes de poder hacer merge me dice esto y al hacer el commit esto otro
Siento tanta molestia pero es la primera vez que uso Github y Git y estoy aprendiendo

@cmaureir
Copy link
Collaborator

Ya he hecho lo que me dices, y he creado el archivo dentro de la carpeta dictionaries pero antes de poder hacer merge me dice esto y al hacer el commit esto otro
Siento tanta molestia pero es la primera vez que uso Github y Git y estoy aprendiendo

Mejor hacer un git status y ver si hay algún archivo que no ha sido agregado, por ejemplo tu diccionario deberia aparece ahí, luego deberías mirar el contenido de eso y agregar el archivo con git add tu_archivo y luego se hace el git commit, mira, te lo muestro acá con un ejemplo en un repositorio local:

🔥 Stargate:~/repo_test  % git status
On branch master

No commits yet

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        super_archivo.txt

nothing added to commit but untracked files present (use "git add" to track)
🥑 Stargate:~/repo_test  % git add super_archivo.txt 
🥑 Stargate:~/repo_test  % git commit -m 'Agregando super_archivo'
[master (root-commit) a5384e1] Agregando super_archivo
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 super_archivo.txt
🥑 Stargate:~/repo_test (master|✅) % git status
On branch master
nothing to commit, working tree clean
🥑 Stargate:~/repo_test (master|✅) % git push origin nombre_de_mi_branch

@cmaureir
Copy link
Collaborator

@Buzzeitor30 como vas? te puedo ayudar en algo?

@cmaureir cmaureir changed the title Traducido archivo cmath.po Traducido library/cmath.po Sep 17, 2020
@cmaureir
Copy link
Collaborator

Arreglado los problemas, este archivo necesita revisión.

@Buzzeitor30
Copy link
Contributor Author

Perdón por responder tan tarde, ¿que ha pasado exactamente?
Si aún no está intentaré hacerlo esta semana

@cmaureir
Copy link
Collaborator

Perdón por responder tan tarde, ¿que ha pasado exactamente?
Si aún no está intentaré hacerlo esta semana

Arreglé todos los problemas, y ahora el PR está listo para ser revisado :) solo eso.
Ahora hay que esperar a algún revisor para que podamos hacer un merge pronto.

Copy link
Contributor

@narvmtz narvmtz left a comment

Choose a reason for hiding this comment

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

¡Muchas gracias por la traducción @Buzzeitor30!

Dejo varias sugerencias, de palabras que tenemos estandarizadas como "return" -> "retorna" y un par de sugerencias de forma.

Creo que resolviendo esto y pasando los checks estaríamos muy cerca del merge 🎉.

@cmaureir
Copy link
Collaborator

Gracias @narvmtz ! si es que pasa la build, avisame si encuentras alguna otra cosa o si lo apruebas para hacer el merge :)

Copy link
Contributor

@narvmtz narvmtz left a comment

Choose a reason for hiding this comment

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

Creo que con esto ya estaríamos listos para el merge y ¡muchas gracias @cmaureir por las modificaciones al archivo!

Copy link
Contributor

@narvmtz narvmtz left a comment

Choose a reason for hiding this comment

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

Y una última pequeña sugerencia.

cmaureir and others added 2 commits September 26, 2020 02:14
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

gracias @narvmtz a ver que dice Travis

@cmaureir
Copy link
Collaborator

Muchas gracias @Buzzeitor30 por la traducción inicial! y gracias @narvmtz por la revisión 🎉

@cmaureir cmaureir merged commit 13a258e into python:3.8 Sep 26, 2020
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 library/cmath.po
3 participants