Skip to content

Actualizado pospell a v1.0.12 #1065

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Aug 3, 2021
Merged

Actualizado pospell a v1.0.12 #1065

merged 22 commits into from
Aug 3, 2021

Conversation

mondeja
Copy link
Contributor

@mondeja mondeja commented Oct 11, 2020 •
edited
Loading

Closes #1061

  • He actualizado la versión de pospell a la 1.0.11. Esta versión produce muchos más mensajes de error, como puedes ver si compruebas la cantidad de términos que he añadido a los diccionarios.
  • He aprovechado para corregir los términos incorrectos que no fueron descubiertos por las versiones anteriores de pospell. Para cada término he ido comprobando si era un posible error de pospell, hunspell, error en el término o en la propia traducción y he ido editando en concordancia o añadiendo a los diccionarios, aunque no dudo que alguno se me habrá escapado.
  • También he eliminado el argumento --modified de pospell, ya que pre-commit, por defecto, sólo se ejecuta sobre los archivos modificados.

@mondeja mondeja marked this pull request as draft October 11, 2020 19:22
@cmaureir
Copy link
Collaborator

cualquier cosa @mondeja estamos en telegram por si quieres discutir más detalles sobre la actualización o alguna otro cambio que quieras hacer ;)
Gracias por el trabajo que ya llevas 🎉

@mondeja mondeja changed the title Actualizando pospell a v1.0.6 Actualizando pospell a v1.0.7 Oct 12, 2020
@mondeja mondeja changed the title Actualizando pospell a v1.0.7 Actualizando pospell a v1.0.6 Oct 12, 2020
@mondeja mondeja changed the title Actualizando pospell a v1.0.6 Actualizando pospell a v1.0.9 Oct 12, 2020
@mondeja mondeja changed the title Actualizando pospell a v1.0.9 Actualizando pospell a v1.0.11 Oct 14, 2020
@mondeja mondeja changed the title Actualizando pospell a v1.0.11 Actualizado pospell a v1.0.11 Oct 14, 2020
@mondeja mondeja marked this pull request as ready for review October 14, 2020 08:06
eamanu
eamanu reviewed Oct 17, 2020
View reviewed changes
Copy link
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

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

dictionaries/library_audioop.txt: se creo un archivo vacio?
dictionaries/library_email_message.txt: si esta vacio no lo debemmos borrar?

@mondeja
Copy link
Contributor Author

mondeja commented Oct 17, 2020 •
edited
Loading

dictionaries/library_audioop.txt: se creo un archivo vacio?

Sí, lo he borrado en el último commit, gracias 👍

dictionaries/library_email_message.txt: si esta vacio no lo debemmos borrar?

Lo he renombrado como dictionaries/library_email.message.txt para mantener la consistencia entre los nombres de módulos.

@cmaureir cmaureir added the hacktoberfest-accepted mantainer-approved contribution to hacktoberfest label Oct 24, 2020
@eamanu
Copy link
Contributor

eamanu commented Nov 19, 2020

@mondeja hay unos conflictos. Creo que estaría bueno si podemos meter este PR :) que les parece?

@mondeja
Copy link
Contributor Author

mondeja commented Nov 19, 2020 •
edited
Loading

El primer problema es que tengo que incluir todas las palabras que ha descubierto de los archivos nuevos. Avisen cuando quieran hacer el merge, las incluyo y lo hacemos coordinadamente. El segundo es que la gran mayoría de pulls abiertos fallarán al incluir este cambio, por lo que habría que volver a ejecutar los flujos de TravisCI y actualizar los diccionarios.

@eamanu
Copy link
Contributor

eamanu commented Nov 20, 2020

@mondeja hmm tal vez sea mejor dejarlo para después del sprint de la PyConAr?

@mondeja
Copy link
Contributor Author

mondeja commented Nov 21, 2020

Supongo que sí

@cmaureir cmaureir added enhancement New feature or request and removed hacktoberfest-accepted mantainer-approved contribution to hacktoberfest labels Mar 20, 2021
@cmaureir cmaureir changed the base branch from 3.8 to 3.9 August 2, 2021 18:35
@cmaureir cmaureir changed the title Actualizado pospell a v1.0.11 Actualizado pospell a v1.0.12 Aug 3, 2021
@cmaureir
Copy link
Collaborator

cmaureir commented Aug 3, 2021

localmente funciona...no sé que estaría faltando en el CI para que pase, a ver ahora.

@cacrespo cacrespo merged commit 2cdadc9 into python:3.9 Aug 3, 2021
cmaureir added a commit that referenced this pull request Aug 4, 2021
@mondeja @cmaureir
Actualizado pospell a v1.0.12 (#1065)
f34d536 
* Actualizando pospell a v1.0.6 (quedan 917 de ~1500 errores) [CI SKIP]

* Actualizando pospell a v1.0.6 (quedan 570 de ~1500 errores) [CI SKIP]

* Actualizando pospell a v1.0.6 (quedan ~366 de ~1500 errores) [CI SKIP]

* Actualizado pospell a versión 1.0.9

* Corregidas excepciones encontradas en TravisCi

* Corregidas excepciones encontradas en TravisCI

* Actualizado pospell a version 1.0.11

* Actualizado con rama 3.8 y eliminado diccionario inútil.

* Añadido 'box' a diccionario.

* Actualizado URL de repositorio pospell en configuración pre-commit

* Actualizando a pospell 1.0.12

* Ajustando versión 1.0.12 y arreglando palabras faltantes

* Agregando palabras faltantes

* Corregir palabra

Co-authored-by: Cristián Maureira-Fredes <Cristian.Maureira-Fredes@qt.io>
@mondeja mondeja deleted the pospell-v1.0.6a1 branch August 7, 2021 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eamanu eamanu eamanu left review comments

@cmaureir cmaureir cmaureir approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Actualizar pospell a próxima versión
4 participants
@mondeja @cmaureir @eamanu @cacrespo