Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Traducido archivo library/rlcompleter #1141

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 6 commits into from
Nov 17, 2020
Merged

Traducido archivo library/rlcompleter #1141

merged 6 commits into from
Nov 17, 2020

Conversation

jaumemy
Copy link
Contributor

@jaumemy jaumemy commented Nov 10, 2020

Closes #817

@jaumemy
Copy link
Contributor Author

jaumemy commented Nov 10, 2020

En cuanto al término "Complete" lo he traducido como finalizar en la mayoría de casos ( como he visto en otra traducción ) excepto en uno donde tenía más sentido para mi la traducción completar

@jaumemy
Copy link
Contributor Author

jaumemy commented Nov 10, 2020

Parece ser que da algún tipo de error con pospell pero al intentar ejecutar el comando me sale el siguiente error:
Can't open affix or dictionary files for dictionary named "fr".

@cmaureir
Copy link
Collaborator

Parece ser que da algún tipo de error con pospell pero al intentar ejecutar el comando me sale el siguiente error:
Can't open affix or dictionary files for dictionary named "fr".

Lo bueno es que puedes ver el error sin tener que ejecutar nada localmente, solo le haces click a "Details" en la fila donde está lo de Travis CI - Pull Requests y luego "The build" de donde puedes ver lo siguiente:

$ pospell -p dict.txt -l es_AR -l es_ES **/*.po
library/rlcompleter.po:100:via
The command "pospell -p dict.txt -l es_AR -l es_ES **/*.po" exited with 255.

Es decir la palabra via no es reconocida, porque le falta un tilde y debería ser vía

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

Muy buena la traducción @jaumemy !
solo encontré un par de espacios en blancos duplicados y dejé un pequeño comentario con el título de los objetos finalizadores.

Copy link
Contributor

@mondeja mondeja left a comment

Choose a reason for hiding this comment

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

Hola de nuevo @jaumemy. Te he dejado una revisión completa 👍

Lo más importante de esta traducción es definir el contexto de "completer", te lo dejo en la primera sugerencia a ver qué te parece. El fallo que marca pospell es la palabra "via" sin tilde en la í, pero si aplicas la sugerencia verás que yo la he sustituido por "por medio de", aunque ya como tú veas, ambas expresiones son válidas.

@jaumemy
Copy link
Contributor Author

jaumemy commented Nov 11, 2020

@cmaureir ¿Me falta por hacer algún cambio que no se ver ?

@mondeja
Copy link
Contributor

mondeja commented Nov 11, 2020

Perfecto @jaumemy 👍 Sólo faltaría añadir al archivo dict que está en la raíz del repositorio. No olvides hacer un commit sólo añadiendo ese archivo para no incluir otros en este pull:

git add dict
git commit -m "Añadida excepción en dict"
git push origin traduccion-rlcompleter

@jaumemy
Copy link
Contributor Author

jaumemy commented Nov 11, 2020

Hecho @mondeja . No sabía que existía el archivo dict para palabras fuera del diccionario :)

@mondeja
Copy link
Contributor

mondeja commented Nov 11, 2020

Hecho @mondeja . No sabía que existía el archivo dict para palabras fuera del diccionario :)

Si el archivo a traducir contiene más palabras, se almacena en la carpeta dictionaries un archivo para esa traducción. Por ejemplo, para este archivo hubiera sido dictionaries/library_rcompleter.txt, pero como sólo hay un término a excluir no merece la pena crear un archivo. 😉

@jaumemy
Copy link
Contributor Author

jaumemy commented Nov 11, 2020

Genial. Gracias por la explicación @mondeja

@jaumemy jaumemy requested a review from cmaureir November 16, 2020 18:02
@cmaureir cmaureir merged commit 6df447f into python:3.8 Nov 17, 2020
@cmaureir
Copy link
Collaborator

Muchas gracias @jaumemy 🎉 💪

@jaumemy jaumemy deleted the traduccion-rlcompleter branch November 17, 2020 11:45
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/rlcompleter.po'
4 participants