- Notifications
You must be signed in to change notification settings - Fork395
Agregar script para crear dict.txt#1059
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Usuarios en Windows, que no utilizan Git bash no pueden generar elarchivo `dict.txt`, pues no tienen acceso al comando 'awk'.Si bien, la construcción de toda la documentación no es necesaria,este paso es importante incluso cuando se quiere hacer la verificaciónpospell a un archivo determinado, pues necesitamos el diccionariogeneral que incluye todas las variaciones de 'dictionaries/'y 'dict'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
¿Qué te parecen estos pequeños cambios en el script?
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
En realidad deberíamos usar |
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Tengo que testear en Windows y macOS y lo convierto a 'ready for review' |
Bgeninatti left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
No soy muy entendido sobre la parte de Travis, pero hice un comentario sobre el script de python.
Gracias por el aporte!
| # Read custom dictionaries | ||
| for filename in Path("dictionaries").glob("*.txt"): | ||
| with open(filename, "r") as f: | ||
| lines = [i.rstrip() for i in f.readlines()] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Que te parece hacerlo case insensitive?
y hacer
lines = [i.rstrip().lower() for i in f.readlines()]There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Lo habíamos discutido antes de implementar lo dedictionaries/ pero al final el tema fue que si algo decía por ejemplo lathe PythonClassImportantBla y alguien lo deja comola pythonclassimportantbla no iba a dejar un error, entonces lo mismo ocurre cuando algo que comienza despues de un punto o un párrafo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Me parece que está perfecto 👍
Usuarios en Windows, que no utilizan Git bash no pueden generar el
archivo
dict.txt, pues no tienen acceso al comando 'awk'.Si bien, la construcción de toda la documentación no es necesaria,
este paso es importante incluso cuando se quiere hacer la verificación
pospell a un archivo determinado, pues necesitamos el diccionario
general que incluye todas las variaciones de 'dictionaries/'
y 'dict'.