-
Notifications
You must be signed in to change notification settings - Fork 396
Añade script check_spell.py #1669
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
La versión anterior de este script cargaba los archivos completos antes de agregarlos al set, lo que usa más memoria de lo requerido. Esta operación se puede realizar más eficientemente usando generadores, con lo cual los archivos se procesan línea a línea. El script también innecesariamente trataba de remover el string vacío del set resultando, lo cual se puede prevenir al momento de agregar elementos al set. Finalmente, el docstring todavía tenía referencias al antiguo archivo "dict", las cuales fueron quitadas. El docstring también fue cambiado de posición de tal manera que aparezca antes de los imports. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
Los usuarios que quieren realizar un chequeo ortográfico sobre los archivos .po actualmente tienen que realizar dos pasos: invocar el sript create_dict.py, el cual genera un archivo dict.txt (un diccionario que contiene a todos los diccionarios dentro de dictionaries/), para luego invocar pospell usando este diccionario generado. Este commit toma el script create_dict.py y le agrega la funcionalidad de invocar pospell luego de generar el diccionario dict.txt (que ahora se genera como un archivo temporal). Con esto, el script ahora cumple la función completa de relizar el chequeo de ortografía, por lo que su nombre ha cambiado a check_spell.py Si no se entregan argumentos, el script corrige todos los archivos .po del repositorio; en caso contrario el usuario debe especificar arhivos .po a ser chequeados. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
…spell.py En el caseo del workflow de GitHub y del Makefile, el cambio es simple: basta con sustituir la invocación a los dos scripts anteriores por el nuevo y ya todo funciona. En el caso de pre-commit, podemos sacar el hook que corre pospell directamente, y en cambio declrar el paquete pospell como una dependencia aditional del nuestro hook local que ahora corre check_spell.py directamente. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
Ya no es necesario que los usuarios realizen dos pasos por separado, sino que ahora solo necesitan correr un solo script. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
cacrespo
approved these changes
Dec 2, 2021
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.
Yo lo veo super bien! Pero no me da la nafta para hacer el merge 😅
tal vez @cmaureir?
cmaureir
approved these changes
Dec 6, 2021
Muchas gracias @rtobar 🎉 |
cmaureir
pushed a commit
that referenced
this pull request
Dec 7, 2021
* Mejora varios aspectos del script create_dict La versión anterior de este script cargaba los archivos completos antes de agregarlos al set, lo que usa más memoria de lo requerido. Esta operación se puede realizar más eficientemente usando generadores, con lo cual los archivos se procesan línea a línea. El script también innecesariamente trataba de remover el string vacío del set resultando, lo cual se puede prevenir al momento de agregar elementos al set. Finalmente, el docstring todavía tenía referencias al antiguo archivo "dict", las cuales fueron quitadas. El docstring también fue cambiado de posición de tal manera que aparezca antes de los imports. * Mueve create_dict.py -> check_spell.py Los usuarios que quieren realizar un chequeo ortográfico sobre los archivos .po actualmente tienen que realizar dos pasos: invocar el sript create_dict.py, el cual genera un archivo dict.txt (un diccionario que contiene a todos los diccionarios dentro de dictionaries/), para luego invocar pospell usando este diccionario generado. Este commit toma el script create_dict.py y le agrega la funcionalidad de invocar pospell luego de generar el diccionario dict.txt (que ahora se genera como un archivo temporal). Con esto, el script ahora cumple la función completa de relizar el chequeo de ortografía, por lo que su nombre ha cambiado a check_spell.py Si no se entregan argumentos, el script corrige todos los archivos .po del repositorio; en caso contrario el usuario debe especificar arhivos .po a ser chequeados. * Cambia GitHub workflows, Makefile y pre-commit hooks para usar check_spell.py En el caseo del workflow de GitHub y del Makefile, el cambio es simple: basta con sustituir la invocación a los dos scripts anteriores por el nuevo y ya todo funciona. En el caso de pre-commit, podemos sacar el hook que corre pospell directamente, y en cambio declrar el paquete pospell como una dependencia aditional del nuestro hook local que ahora corre check_spell.py directamente. * Actualiza FAQ sobre cómo hacer chequeo de ortografía Ya no es necesario que los usuarios realizen dos pasos por separado, sino que ahora solo necesitan correr un solo script. Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
El nuevo script
check_spell.py
consiste en el antiguocreate_dict.py
más un llamado automático apospell
. Todas las referencias al combocreate_dict.py
+pospell
han sido eliminadas, y en su lugar ahora se llama simplemente acheck_spell.py
.check_spell.py
sin argumentos chequea todo el repositorio (como se usa en el worfklow de GitHub y en el Makefile). Si se entregan argumentos, se chequea sólo esos archivos (como se usa con pre-commit).