Skip to content

Homogeneizar las traducciones de // y floor division a "división entera a la baja" #2797

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
14 tasks done
jcaalzago opened this issue Jan 17, 2024 · 10 comments
Closed
14 tasks done
Assignees

Comments

@jcaalzago
Copy link
Contributor

jcaalzago commented Jan 17, 2024

Como motivo de la votación realizada en telegram en la que se consideraba el cambio en la traducción de floor division y // propuesto en la issue 2754, en la que el resultado de la misma fue:

imagen

Realizo un listado de los ficheros en los que se hace mención a estos términos (puede sufrir algún cambio, en caso que falte o sobre alguno):

El término de floor division en el módulo decimal es distinto, y se acordó no tocarlo en este momento.

@MateoCamara
Copy link
Contributor

Venga, yo me puedo encargar. Esta sería mi segunda contribución.

@cacrespo
Copy link
Collaborator

👏 👏 👏
Muy bueno el listado con los files a trabajar @jcaalzago. Lo retoqué un poquito.

Les recuerdo el paso a paso para contribuir está aquí

¿Les parece si vamos mencionando en el primer comentario los files que vamos trabajando? (me adelanté y les asigné 2 a cada uno :P)

¡Muy buen trabajo!

@jcaalzago
Copy link
Contributor Author

jcaalzago commented Jan 20, 2024

@cacrespo ¿Creamos una PR por cada fichero?

@cacrespo
Copy link
Collaborator

@cacrespo ¿Creamos una PR por cada fichero?

La verdad que es lo mismo... son cambios tan chiquitos que tal vez vale la pena juntar varios en un sólo PR y ya.
Acá corregí los últimos dos. Si alguno puede hacer el review así mergeamos sería estupendo.

@jcaalzago
Copy link
Contributor Author

@cacrespo ¿Creamos una PR por cada fichero?

La verdad que es lo mismo... son cambios tan chiquitos que tal vez vale la pena juntar varios en un sólo PR y ya. Acá corregí los últimos dos. Si alguno puede hacer el review así mergeamos sería estupendo.

Creo que ya la revisé (es la primera vez que hago revisión).
También he creado 3 PR para el resto de ficheros que quedaban pendientes (menos la memoria de traducción).

Cuando podáis revisadme las PR.

mmmarcos pushed a commit that referenced this issue Jan 26, 2024
cacrespo added a commit that referenced this issue Jan 27, 2024
Part of #2797

---------

Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
mmmarcos pushed a commit that referenced this issue Jan 27, 2024
cacrespo added a commit that referenced this issue Jan 28, 2024
Part of #2797

---------

Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
cacrespo pushed a commit that referenced this issue Jan 28, 2024
@cacrespo
Copy link
Collaborator

cacrespo commented Jan 30, 2024

Hola @MateoCamara, ¿has podido avanzar con los ajustes que tenías asignados? Sólo nos faltan esos dos y terminamos 🥳 .

@MateoCamara
Copy link
Contributor

Muy buenas!! No estaba seguro de si finalmente tenía yo asignado parte, sorry! Me pongo con ello :)

Copy link

Este issue lleva un tiempo sin actualizaciones. ¿Estás trabajando todavía?
Si necesitas ayuda 🆘 no dudes en contactarnos en nuestro grupo de Telegram.

@github-actions github-actions bot added the stale label Feb 10, 2024
@jcaalzago
Copy link
Contributor Author

Muy buenas!! No estaba seguro de si finalmente tenía yo asignado parte, sorry! Me pongo con ello :)

Ya se terminaron todas las tareas de esta issue, ¿no? Lo digo para cerrarla

@github-actions github-actions bot removed the stale label Feb 11, 2024
@MateoCamara
Copy link
Contributor

Muy buenas!! No estaba seguro de si finalmente tenía yo asignado parte, sorry! Me pongo con ello :)

Ya se terminaron todas las tareas de esta issue, ¿no? Lo digo para cerrarla

Están terminadas, sí! 👍

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

No branches or pull requests

3 participants