Skip to content

Translate library/socket.po #1371

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 42 commits into from
Sep 17, 2021
Merged

Conversation

andresbermeoq
Copy link
Contributor

Closes #1311

@andresbermeoq
Copy link
Contributor Author

@cacrespo could you help me about the test ?.
I'm running the tests in local, and it doesn't give me any error. Thanks

@cacrespo
Copy link
Collaborator

cacrespo commented Sep 7, 2021

Hola @andresbermeoq! Podemos conversarlo en español, no hay problema. 😄
El test falla porque pospell no encuentra algunas palabras en el diccionario por defecto.
Si vas a "Details" te aparece el output con la información del fallo.

Estas son las palabras en cuestión:

library/socket.po:174:modulos
library/socket.po:174:ambito
library/socket.po:287:parametros
library/socket.po:287:direccion
library/socket.po:321:Bluetooth
library/socket.po:329:Bluetooth
library/socket.po:337:Bluetooth
library/socket.po:337:Bluetooth
library/socket.po:353:Bluetooth
library/socket.po:487:permítala
library/socket.po:599:defínalas
library/socket.po:872:Bluetooth
library/socket.po:893:enrutador
library/socket.po:919:talvez
library/socket.po:941:autodetectados
library/socket.po:1154:fileno
library/socket.po:1154:inet
library/socket.po:1354:aliaslist
library/socket.po:1354:ipaddrlist
library/socket.po:1354:gethostbyname
library/socket.po:1354:getaddrinfo
library/socket.po:1669:Talvez
library/socket.po:1708:portadles
library/socket.po:1845:descripcion
library/socket.po:2075:limitacion
library/socket.po:2180:podrian
library/socket.po:2189:cóntrol
library/socket.po:2605:parametros
library/socket.po:3069:Stuart
library/socket.po:3069:Sechrest
library/socket.po:3077:Leffler
library/socket.po:3085:Winsock

Algunas son faltas ortográficas y es cuestión de corregirlas y ya. Hay otras que evidentemente son nombres propios (ej. Sechrest, Leffler, etc.). En ese caso es necesario agregar un archivo que se llame library_socket.txt en la carpeta /dictionaries con una palabra por linea.
Prueba haciendo eso y seguramente pase todo OK. Si sigue fallando exploramos un poco más qué puede estar pasando y lo resolvemos. 💪

@andresbermeoq
Copy link
Contributor Author

Que tal @cacrespo … Listo ya no me da ningún error de ortografía, pero Winsock me sigue dando a pesar de que ya le agregue a library_socket.txt

@cacrespo
Copy link
Collaborator

cacrespo commented Sep 8, 2021

Genial! Voy a tratar de revisar el archivo entre hoy y mañana. 😄

@andresbermeoq
Copy link
Contributor Author

Ya logré solucionar... ha sido un problema de mayúsculas y minúsculas..

Copy link
Collaborator

@cacrespo cacrespo left a comment

Choose a reason for hiding this comment

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

Va revisión hasta linea 1000.

¡Tremendo trabajo has hecho @andresbermeoq!

andresbermeoq and others added 11 commits September 10, 2021 10:44
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
@andresbermeoq
Copy link
Contributor Author

Disculpa @cacrespo talvez pudieras ayudar con el feedback del error. Solamente le cambie las palabras

@cacrespo
Copy link
Collaborator

cacrespo commented Sep 10, 2021

Hola @andresbermeoq !

Disculpa @cacrespo talvez pudieras ayudar con el feedback del error. Solamente le cambie las palabras

¿Te refieres a lo de WinSock y Winsock?

En este hilo se desarrolla el tema: #1352

Básicamente cuando pospell realiza el chequeo ortográfico utiliza una aplicación que se llama hunspell. Esta última crea el diccionario que hace el chequeo pero lo hace sin un orden específico en el listado de las palabras. De modo que es medio aleatorio según qué palabra quede primero se produce un problema o no.

¿o te refieres a otra cosa? Ahh no. Ya entendí a qué te referías. 😄

Lo que está pasando con los nuevos ajustes es que se rompió el formato del archivo .po (estos archivos tienen que respetar un ancho determinado por ej.).
Hay dos formas de solucionarlo:

  • Instalas la herramienta powrap y ejecutas el comando powrap socket.po
  • Si usas poedit para traducir, simplemente actualiza tu copia local con todos los cambios que acabas de impactar y luego abre el archivo y vuelve a guardar.

Trata de hacerlo y si se complica avisa. Podemos seguirlo en el canal de Telegram si te resulta más práctico.

andresbermeoq and others added 17 commits September 16, 2021 18:11
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
andresbermeoq and others added 2 commits September 17, 2021 12:32
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
@andresbermeoq
Copy link
Contributor Author

Que tal @cacrespo... disculpa el diccionario con la palabra util lleva tilde según el diccionario. Talvez por eso no está pasando el test.

@cacrespo
Copy link
Collaborator

cacrespo commented Sep 17, 2021

Que tal @cacrespo... disculpa el diccionario con la palabra util lleva tilde según el diccionario. Talvez por eso no está pasando el test.

Tienes razón!!!
Siempre me confunde y se me hacía que iba sin tilde. 🤦🤦🤦

Pasale util y ya mergeamos 😄

@andresbermeoq
Copy link
Contributor Author

Muchas gracias por tu tiempo @cacrespo... tomaré otro issue en las próximas semanas

@cacrespo cacrespo merged commit ce2983a into python:3.9 Sep 17, 2021
@cacrespo
Copy link
Collaborator

Felicitaciones @andresbermeoq!!! 🥳🥳🥳

cmaureir pushed a commit that referenced this pull request Dec 7, 2021
* Translate the library/socket.po
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/socket.po'
2 participants