Skip to content

Traduccion archivo faq/gui.po#209

Merged
clacri merged 22 commits into
python:3.8from
pyjavo:traduccion-faq/gui
May 11, 2020
Merged

Traduccion archivo faq/gui.po#209
clacri merged 22 commits into
python:3.8from
pyjavo:traduccion-faq/gui

Conversation

@pyjavo

@pyjavo pyjavo commented May 9, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@humitos humitos 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 bueno! Gracias. Hice algunas sugerencias 😄

Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
pyjavo and others added 15 commits May 11, 2020 14:29
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
@pyjavo

pyjavo commented May 11, 2020

Copy link
Copy Markdown
Contributor Author

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

@raulcd raulcd 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.

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

raulcd commented May 11, 2020

Copy link
Copy Markdown
Collaborator

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

pyjavo commented May 11, 2020

Copy link
Copy Markdown
Contributor Author

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

raulcd commented May 11, 2020

Copy link
Copy Markdown
Collaborator

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

pyjavo commented May 11, 2020

Copy link
Copy Markdown
Contributor Author

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

@humitos

humitos commented May 11, 2020

Copy link
Copy Markdown
Collaborator

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

@clacri clacri 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 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 :)

Comment thread faq/gui.po Outdated
Comment thread faq/gui.po Outdated
Co-authored-by: Claudia Millán <clmilneb@gmail.com>
Co-authored-by: Claudia Millán <clmilneb@gmail.com>
@clacri clacri merged commit 7753fc0 into python:3.8 May 11, 2020
@clacri

clacri commented May 11, 2020

Copy link
Copy Markdown
Contributor

Closes #162

@humitos

humitos commented May 12, 2020

Copy link
Copy Markdown
Collaborator

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