Skip to content
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

Separador basado en tex-hyphen-spanish #343

Merged
merged 7 commits into from
Dec 30, 2024
Merged

Separador basado en tex-hyphen-spanish #343

merged 7 commits into from
Dec 30, 2024

Conversation

cosmoscalibur
Copy link
Collaborator

@cosmoscalibur cosmoscalibur commented Dec 27, 2024

Considerando los reportes del separador y la existencia del proyecto tex-hyphen-spanish de @jbezos, con mayor cobertura de casos que el actual en RLA-ES, se implementa la generación de las reglas del separador basado en este.

Descripción de cambios:

  • Rutina para actualización de patrones de silabación a partir de tex-hyphen-spanish.
  • Actualización de ISO 8859-1 a UTF8 y Python2 a Python3 en rutina de prueba (primera parte en master 02c2abc).
  • Ajuste de ejemplos al español de la rutina de pruebas (primera parte en master 02c2abc y nuevo ajuste basado en tex-hyphen-spanish en el PR).
  • Actualización de README.
  • Actualización de texto de licencia para reflejar el fuente en MIT y la distribución general en el esquema ya existente.

Al revisarse en detalle, no hay problema con el reemplazo de la licencia del fichero, porque estrictamente la licencia nunca aplicó para el fichero (se hace mención a un archivo inexistente, hyph_es_ANY), pero es claro que la distribución como un todo debe ser en el esquema existente.

Este PR reemplaza #316
Soluciona #205, muchos de #264 y la inclusión propuesta en #270 basada en tex-hyphen-spanish.
No es incompatible con #196 porque 'de-sa-yu-no' también es válido.

@cosmoscalibur cosmoscalibur self-assigned this Dec 27, 2024
@cosmoscalibur
Copy link
Collaborator Author

@RickieES @jbezos , si tienen alguna observación con los ajustes de los textos de README y licencia.
Se aprovechó a otras actualizaciones que describo al inicio. La idea sería poder fusionar esto para la versión próxima del diccionario.

@jbezos
Copy link

jbezos commented Dec 28, 2024

👍 Adelante.

@cosmoscalibur cosmoscalibur merged commit a44f867 into master Dec 30, 2024
@cosmoscalibur cosmoscalibur deleted the separador-tex branch December 30, 2024 15:42
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