Skip to content

Dejar solo es_ES como diccionario #960

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 7 commits into from
Dec 19, 2020
Merged

Conversation

cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Oct 5, 2020

Actualmente 'make spell' verifica todo localmente
utilizando solo el diccionario es_ES, pero en la configuración
de travis, estamos usando también es_AR.

Para tener consistencia, sugiero que sería mejor dejar solo
es_ES por defecto en ambos lados y aceptar las variaciones
del lenguaje para cada pais mediante los diccionarios de cada archivo.

Actualmente 'make spell' verifica todo localmente
utilizando solo el diccionario es_ES, pero en la configuración
de travis, estamos usando también es_AR.

Para tener consistencia, sugiero que sería mejor dejar solo
es_ES por defecto en ambos lados y aceptar las variaciones
del lenguaje para cada pais mediante los diccionarios de cada archivo.
@Letram
Copy link
Contributor

Letram commented Oct 6, 2020

Yo no lo vería mal. Además, el uso de diccionarios es bastante sencillo

@clacri
Copy link
Contributor

clacri commented Oct 6, 2020

Pues como veáis, porque no se los números pero al menos previo al hacktober me daba l sensación de que teníamos muchas contribuciones de Argentina, pero vamos para mí es lo mismo :)

@cmaureir
Copy link
Collaborator Author

cmaureir commented Oct 7, 2020

@humitos alguna opinión acá?

@clacri
Copy link
Contributor

clacri commented Oct 11, 2020

Si al final lo cambiamos entonces también modificaría el configurador de pre-commit porque este lo usa también

@cmaureir
Copy link
Collaborator Author

Si al final lo cambiamos entonces también modificaría el configurador de pre-commit porque este lo usa también

done

@cmaureir cmaureir added the hacktoberfest-accepted mantainer-approved contribution to hacktoberfest label Oct 24, 2020
@@ -0,0 +1 @@
testear
Copy link
Collaborator

Choose a reason for hiding this comment

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

Estás y todas las que siguen, entiendo que quedaron pegadas de alguna traducción ¿no?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Claro, solo generé hace tiempo 1 diccionario por cada archivo y quizás algunas fueron cambiadas o están en otros lados. Agregué tus sugerencias a ver que dice Travis.

@python python deleted a comment from cacrespo Nov 23, 2020
Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
@cmaureir
Copy link
Collaborator Author

cmaureir commented Dec 3, 2020

hay un par de archivos vacios que tengo que borrar y mergeamos

@cmaureir
Copy link
Collaborator Author

@cacrespo @eamanu solo borré un par de archivos vacios, asi que asumo que sus aprobaciones se mantienen :)

@cmaureir cmaureir merged commit 208236e into python:3.8 Dec 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted mantainer-approved contribution to hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants