Skip to content

Mueve dict -> dictionaries/_base.txt #1651

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 1 commit into from
Nov 25, 2021
Merged

Conversation

rtobar
Copy link
Collaborator

@rtobar rtobar commented Nov 24, 2021

El archivo dict anteriormente era el único archivo donde se guardaban
las palabras para usar con el chequeador de ortografía, Actualmente
usamos la técnica de tener un archivo por módulo dentro del
ubdirectorio diccionaries/ donde se agregan las palabras de forma
individual al módulo que corresponde. Al momento de chequear la
ortografía del proyecto entero, se mergean todos los diccionarios en uno
solo llamado dict.txt, el cual existe efímeramente, y que no tiene que
ser editado.

La existencia de dos archivos dict y dict.txt es confusa, y más aún
cuando hay además un subdirectorio con más diccionarios dentro. Para
mejorar esta situación, este commit mueve dict a
dictionaries/_base.txt. El nunevo nombre sirve dos propósitos: la
extensión .txt asegura que el diccionario va a ser parte del proceso de
mergeo de diccionarios; y segundo, el prefijo _ lo hace ver como un
archivo especial que se diferencia claramente del resto.

Finalmente, también aproveché de sacar el target dict_dups del Makefile,
el cual no tenía sentido mantener.

Signed-off-by: Rodrigo Tobar rtobar@icrar.org

El archivo "dict" anteriormente era el único archivo donde se guardaban
las palabras para usar con el chequeador de ortografía, Actualmente
usamos la técnica de tener un archivo por módulo dentro del
ubdirectorio diccionaries/ donde se agregan las palabras de forma
individual al módulo que corresponde. Al momento de chequear la
ortografía del proyecto entero, se mergean todos los diccionarios en uno
solo llamado "dict.txt", el cual existe efímeramente, y que no tiene que
ser editado.

La existencia de dos archivos "dict" y "dict.txt" es confusa, y más aún
cuando hay además un subdirectorio con más diccionarios dentro. Para
mejorar esta situación, este commit mueve "dict" a
diccionaries/_base.txt. El nunevo nombre sirve dos propósitos: la
extensión .txt asegura que el diccionario va a ser parte del proceso de
mergeo de diccionarios; y segundo, el prefijo "_" lo hace ver como un
archivo especial que se diferencia claramente del resto.

Finalmente, también aproveché de sacar el target dict_dups del Makefile,
el cual no tenía sentido mantener.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
@cmaureir cmaureir merged commit 059f834 into python:3.10 Nov 25, 2021
@rtobar rtobar deleted the diccinoarios branch November 30, 2021 03:08
cmaureir pushed a commit that referenced this pull request Dec 7, 2021
El archivo "dict" anteriormente era el único archivo donde se guardaban
las palabras para usar con el chequeador de ortografía, Actualmente
usamos la técnica de tener un archivo por módulo dentro del
ubdirectorio diccionaries/ donde se agregan las palabras de forma
individual al módulo que corresponde. Al momento de chequear la
ortografía del proyecto entero, se mergean todos los diccionarios en uno
solo llamado "dict.txt", el cual existe efímeramente, y que no tiene que
ser editado.

La existencia de dos archivos "dict" y "dict.txt" es confusa, y más aún
cuando hay además un subdirectorio con más diccionarios dentro. Para
mejorar esta situación, este commit mueve "dict" a
diccionaries/_base.txt. El nunevo nombre sirve dos propósitos: la
extensión .txt asegura que el diccionario va a ser parte del proceso de
mergeo de diccionarios; y segundo, el prefijo "_" lo hace ver como un
archivo especial que se diferencia claramente del resto.

Finalmente, también aproveché de sacar el target dict_dups del Makefile,
el cual no tenía sentido mantener.

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants