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

Skip to content

Traduccion archivo faq/gui.po #209

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 22 commits into from
May 11, 2020
Merged

Traduccion archivo faq/gui.po #209

merged 22 commits into from
May 11, 2020

Conversation

pyjavo
Copy link
Contributor

@pyjavo pyjavo commented May 9, 2020

No description provided.

Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

Muy bueno! Gracias. Hice algunas sugerencias 😄

@pyjavo
Copy link
Contributor Author

pyjavo commented May 11, 2020

hola @humitos @cmaureir
No entiendo el error que tiene travis. ¿Cómo lo corrijo?

Copy link
Collaborator

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Hola, muchísimas gracias por el trabajo. Utilizamos un script para que se mantenga el ancho de línea de los archivos. La herramienta que utilizamos es powrap y está fallando:

$ powrap --check --quiet **/*.po
Would rewrap:
- faq/gui.po
The command "powrap --check --quiet **/*.po" exited with 1.

También utilizamos pospell para comprobar las palabras en el diccionario y hay algunas que parece no están en nuestro diccionario:

$ pospell -p dict -l es_AR -l es_ES **/*.po
faq/gui.po:58:widgets
faq/gui.po:58:Tcl
faq/gui.po:58:Tcl
faq/gui.po:81:Ruby
faq/gui.po:95:wxPython
faq/gui.po:110:wxPython
faq/gui.po:124:PySide
faq/gui.po:124:PySide
faq/gui.po:167:PyGtk
faq/gui.po:167:Henstridge
faq/gui.po:180:Android
faq/gui.po:180:Cython
faq/gui.po:192:Kivy
faq/gui.po:230:Cocoa
faq/gui.po:238:Mark
faq/gui.po:238:Hammond
faq/gui.po:238:Microsoft
faq/gui.po:238:Foundation
faq/gui.po:248:Tkinter
faq/gui.po:252:Tkinter
faq/gui.po:256:Tkinter
faq/gui.po:256:Tcl
faq/gui.po:267:Tcl
faq/gui.po:277:Tcl
faq/gui.po:277:Tix
faq/gui.po:289:Tix
faq/gui.po:289:libtclsam
faq/gui.po:289:libtclsam
faq/gui.po:289:libtksam
faq/gui.po:289:Tix
faq/gui.po:304:I
faq/gui.po:304:Xt
faq/gui.po:304:I
faq/gui.po:319:Tkinter
faq/gui.po:333:widget
The command "pospell -p dict -l es_AR -l es_ES **/*.po" exited with 255.

algunas como Tkinter o nombre propios se pueden añadir al archivo dict para que no se tengan en cuenta.

@raulcd
Copy link
Collaborator

raulcd commented May 11, 2020

Para arreglar powrap puedes instalarlo en un entorno virtual de python y ejecutar en el archivo que falla o si quieres lo puedo hacer yo mismo:
powrap faq/gui.po

@pyjavo
Copy link
Contributor Author

pyjavo commented May 11, 2020

Gracias @raulcd. Modificaré el dict e instalaré powrap. En el requirements.txt no hay una versión específica, ¿supongo que pueda usar la más estable?

@raulcd
Copy link
Collaborator

raulcd commented May 11, 2020

powrap pasa perfecto ahora, falta una palabra en pospell (aunque la has escrito de dos formas diferentes) con la mayúscula en Widgets wxWidgets o sin wxwidgets

4.25s$ powrap --check --quiet **/*.po
The command "powrap --check --quiet **/*.po" exited with 0.
$ pospell -p dict -l es_AR -l es_ES **/*.po
faq/gui.po:80:wxWidgets
faq/gui.po:94:wxwidgets
faq/gui.po:94:wxWidgets
faq/gui.po:109:wxWidgets

@pyjavo
Copy link
Contributor Author

pyjavo commented May 11, 2020

Listo. Las agregue al dict. Hay una frase que tiene ambas wxWidgets (https://www.wxwidgets.org)

@humitos
Copy link
Collaborator

humitos commented May 11, 2020

@JavierDaza si querés podes instalar "pre-commit" que ejecuta powrap, pospell y ordena el dict automáticamente antes de hacer un commit

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Hola Javi, solo te he puesto alguna sugerencia, me parece que idioma aplicado a lenguaje de programación queda raro, pero esto puede ser subjetivo. Mira que te parece :)

Co-authored-by: Claudia Millán <[email protected]>
Co-authored-by: Claudia Millán <[email protected]>
@clacri clacri merged commit 7753fc0 into python:3.8 May 11, 2020
@clacri
Copy link
Contributor

clacri commented May 11, 2020

Closes #162

@humitos
Copy link
Collaborator

humitos commented May 12, 2020

Closes #162

@clacri Creo que la única forma en la que funciona es si se pone en la descripción del issue y hay que hacerlo antesd de hacer merge 😄

Lo ideal sería que lo haga la persona que abre el PR 😉

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.

5 participants