Skip to content

Traducido archivo library/rlcompleter - #1141

Merged
cmaureir merged 6 commits into
python:3.8from
jaumemy:traduccion-rlcompleter
Nov 17, 2020
Merged

Traducido archivo library/rlcompleter#1141
cmaureir merged 6 commits into
python:3.8from
jaumemy:traduccion-rlcompleter

Conversation

@jaumemy

@jaumemy jaumemy commented Nov 10, 2020

Copy link
Copy Markdown
Contributor

Closes #817

@jaumemy

jaumemy commented Nov 10, 2020

Copy link
Copy Markdown
Contributor Author

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

jaumemy commented Nov 10, 2020

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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

@cmaureir cmaureir left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated

@mondeja mondeja left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
Comment thread library/rlcompleter.po Outdated
@jaumemy

jaumemy commented Nov 11, 2020

Copy link
Copy Markdown
Contributor Author

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

Comment thread library/rlcompleter.po Outdated
@mondeja

mondeja commented Nov 11, 2020

Copy link
Copy Markdown
Contributor

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

jaumemy commented Nov 11, 2020

Copy link
Copy Markdown
Contributor Author

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

@mondeja

mondeja commented Nov 11, 2020

Copy link
Copy Markdown
Contributor

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

jaumemy commented Nov 11, 2020

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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