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

Traducido archivo library/rlcompleter #1141

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

@cmaureir cmaureir 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".

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

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.

library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
Copy link
Contributor

@mondeja mondeja left a comment

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.

library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
library/rlcompleter.po Outdated Show resolved Hide resolved
jaumemy added 2 commits Nov 10, 2020
library/rlcompleter.po Outdated Show resolved Hide resolved
@jaumemy
Copy link
Contributor Author

@jaumemy jaumemy commented Nov 11, 2020

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

library/rlcompleter.po Outdated Show resolved Hide resolved
@mondeja
Copy link
Contributor

@mondeja 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 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 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 jaumemy commented Nov 11, 2020

Genial. Gracias por la explicación @mondeja

@eamanu
eamanu approved these changes Nov 14, 2020
@jaumemy jaumemy requested a review from cmaureir Nov 16, 2020
@cmaureir cmaureir merged commit 6df447f into python:3.8 Nov 17, 2020
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Nov 17, 2020

Muchas gracias @jaumemy 🎉 💪

@jaumemy jaumemy deleted the jaumemy:traduccion-rlcompleter branch Nov 17, 2020
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.

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