Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
cacrespo merged 22 commits intopython:3.9frommondeja:pospell-v1.0.6a1
Aug 3, 2021
Merged

Conversation

mondeja
Copy link
Contributor

@mondejamondeja commentedOct 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 quepre-commit, por defecto, sólo se ejecuta sobre los archivos modificados.

@mondejamondeja marked this pull request as draftOctober 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 reacted with heart emoji

@mondejamondeja changed the titleActualizando pospell a v1.0.6Actualizando pospell a v1.0.7Oct 12, 2020
@mondejamondeja changed the titleActualizando pospell a v1.0.7Actualizando pospell a v1.0.6Oct 12, 2020
@mondejamondeja changed the titleActualizando pospell a v1.0.6Actualizando pospell a v1.0.9Oct 12, 2020
@mondejamondeja changed the titleActualizando pospell a v1.0.9Actualizando pospell a v1.0.11Oct 14, 2020
@mondejamondeja changed the titleActualizando pospell a v1.0.11Actualizado pospell a v1.0.11Oct 14, 2020
@mondejamondeja marked this pull request as ready for reviewOctober 14, 2020 08:06
Copy link
Contributor

@eamanueamanu 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
ContributorAuthor

mondeja commentedOct 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 comodictionaries/library_email.message.txt para mantener la consistencia entre los nombres de módulos.

@cmaureircmaureir added the hacktoberfest-acceptedmantainer-approved contribution to hacktoberfest labelOct 24, 2020
@eamanu
Copy link
Contributor

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

@mondeja
Copy link
ContributorAuthor

mondeja commentedNov 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

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

@mondeja
Copy link
ContributorAuthor

Supongo que sí

cmaureir reacted with thumbs up emoji

@cmaureircmaureir added enhancementNew feature or request and removed hacktoberfest-acceptedmantainer-approved contribution to hacktoberfest labelsMar 20, 2021
@cmaureircmaureir changed the base branch from3.8 to3.9August 2, 2021 18:35
@cmaureircmaureir changed the titleActualizado pospell a v1.0.11Actualizado pospell a v1.0.12Aug 3, 2021
@cmaureir
Copy link
Collaborator

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

mondeja reacted with thumbs up emoji

@cacrespocacrespo merged commit2cdadc9 intopython:3.9Aug 3, 2021
cmaureir added a commit that referenced this pull requestAug 4, 2021
* 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 palabraCo-authored-by: Cristián Maureira-Fredes <Cristian.Maureira-Fredes@qt.io>
@mondejamondeja deleted the pospell-v1.0.6a1 branchAugust 7, 2021 15:03
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@eamanueamanueamanu left review comments

@cmaureircmaureircmaureir approved these changes

Assignees
No one assigned
Labels
enhancementNew 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
[8]ページ先頭
©2009-2025 Movatter.jp