Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Chequea orden en TRANSLATORS #1830

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 2 commits into from
Jul 10, 2022
Merged

Conversation

rtobar
Copy link
Collaborator

@rtobar rtobar commented Jul 10, 2022

Dado que el archivo TRANSLATORS fue ordenado alfabéticamente, sería bueno mantener ese orden en el futuro.

Este PR agrega un paso en el GitHub actions para chequear que el archivo sigue estando ordenado alfabéticamente, y genera un error de no ser así. El chequeo se hace comparando el archivo sin modificar contra la salida de un nuevo script scripts/sort.py, el cual lee el archivo, ordena las entradas alfabéticamente, e imprime el resultado. Si el resultado es igual al contenido del archivo entonces el archivo está ya ordenado.

El script es genérico, por lo que puede ser usado en el futuro para otras tareas similares.

El ordenamiento se hace usando el colaje de cadenas de texto del locale
actual, y sobre stdin, haciendo la herramienta lo más flexible posible.

Signed-off-by: Rodrigo Tobar <[email protected]>
@rtobar rtobar force-pushed the chequea-translators branch 2 times, most recently from eb57138 to 3f2372b Compare July 10, 2022 15:17
@rtobar rtobar changed the title Chequea orden en TRANSLATORS Draft: Chequea orden en TRANSLATORS Jul 10, 2022
Cuando nueva gente se agregue a sí misma a la lista de autores en
TRANSLATORS, este nuevo paso en GitHub actions corroborará que el
archivo TRANSLATORS sigue estando ordenado alfabéticamente, y generará
un error de no ser éste el caso.

Signed-off-by: Rodrigo Tobar <[email protected]>
@rtobar rtobar force-pushed the chequea-translators branch from 3f2372b to c68037e Compare July 10, 2022 15:33
@rtobar rtobar changed the title Draft: Chequea orden en TRANSLATORS Chequea orden en TRANSLATORS Jul 10, 2022
@cmaureir
Copy link
Collaborator

Super buena idea :D gracias 🎉

@cmaureir cmaureir merged commit 29d216b into python:3.10 Jul 10, 2022
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