Skip to content

pre-commit fix #661

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

Closed
clacri opened this issue Aug 16, 2020 · 5 comments · Fixed by #701
Closed

pre-commit fix #661

clacri opened this issue Aug 16, 2020 · 5 comments · Fixed by #701
Assignees

Comments

@clacri
Copy link
Contributor

clacri commented Aug 16, 2020

Parece que hay un error con pre-commit y ahora no se corresponde con nuestro building process. No sé nada de como arreglarlo pero estaría genial echarle un vistazo.

@cmaureir
Copy link
Collaborator

el cambio que se hizo a nuestro proceso de crear los docs fue pasar de:

cat dict dictionaries/*.txt > dict.txt
pospell -p dict.txt .....

por

awk 1 dict dictionaries/*.txt > dict.txt
pospell -p dict.txt .....

Solo para evitar el problema cuando algunos archivos no tenian un 'salto de línea' al final de ellos (no confundir con una línea en blanco).

Quizás pre-commit tiene algo interno donde hace ese proceso y se pueda arreglar, quizás podrías ver algun archivo de configuración cuando se instala.

@clacri
Copy link
Contributor Author

clacri commented Aug 16, 2020

El fichero de configuración de pre commit es este https://www.github.com/python/python-docs-es/tree/3.8/.pre-commit-config.yaml pero entiendo que el chequeo de pospell es una llamada a un script/hook como se llame en el repo de humitos. Le echo un vistazo

@clacri
Copy link
Contributor Author

clacri commented Aug 16, 2020

Creo que es más simple, tenemos que cambiar la llamada al pospell en el fichero yaml del pre-commit para que utilice el fichero global no? Ya no es más un dict en esa carpeta sino el generado de nuestro set. Mañana me lo miro en el ordenador que estoy con el móvil y medio dormida.

@humitos humitos self-assigned this Aug 17, 2020
@humitos
Copy link
Collaborator

humitos commented Aug 17, 2020

Me asigno este para arreglarlo cuando tenga un tiempito

@clacri
Copy link
Contributor Author

clacri commented Aug 17, 2020

genial! Sólo por curiosidad, por lo que he entendido viendo tu codigo y el pre-commit config en realidad solo son llamadas a scripts que hay en diferentes repositorios. En el caso de nuestro pre-commit aun estaba haciendo la llamada a pospell con nuestro fichero dict (que generabamos nosotros y siempre estaba creado). Pero ahora tenemos q añadir un paso más. Entonces que hay que hacer, hacer un script q haga lo de awk, ponerlo en un repo y añadir ese hook entre el de powrap y el de pospell?

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 a pull request may close this issue.

3 participants