search

IslasGECI / seleccion_analista_2022_mvb

🐍 Mario Villasante Barahona
GNU Affero General Public License v3.0
0 stars 0 forks source link

Corrije formato de black a --max-line-length 100 #21

Closed mvillasante closed 2 years ago

mvillasante commented 2 years ago

No me dió la cabeza para implementar la separación del dataset en cuatro funciones con una sola salida...

codecov[bot] commented 2 years ago

The author of this PR, mvillasante, is not an activated member of this organization on Codecov. Please activate this user on Codecov to display this PR comment. Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations. Please don't hesitate to email us at support@codecov.io with any questions.

devarops commented 2 years ago

Estábamos a punto de terminar este PR. Te recomiendo que en el futuro resistas la tentación de agregar más código a un PR que está casi terminado. Este PR ahora tiene 87 líneas. Las revisiones te obligarán a incrementar el número de líneas. Rechazaré los PR de 100 líneas.

image

devarops commented 2 years ago

¿Por favor podrías presionar el botón Re-request review cuando estés listo para mi siguiente revisión?

mvillasante commented 2 years ago

No estaba prestando atención a la longitud del PR.... :cold_sweat: Qué podemos hacer al respecto? Si esta es la cuarta función que estoy consignando? Deberíamos llevar 4 PR?

devarops commented 2 years ago

No estaba prestando atención a la longitud del PR.... Qué podemos hacer al respecto?

Podríamos partir este PR en dos PR más pequeños (#22). O nos podemos arriesgar y seguirle con este PR como está. Tal vez no lleguemos a 100 líneas. 🤷🏾 🤞🏾

Si esta es la cuarta función que estoy consignando? Deberíamos llevar 4 PR?

No necesariamente. Si son funciones muy pequeñas puedes meter más de una función en el mismo PR.

mvillasante commented 2 years ago

Acabo de pasar el commit a otra rama.

Podríamos unir esta rama?

devarops commented 2 years ago

Acabo de pasar el commit a otra rama.

Podríamos unir esta rama?

¡Buena idea! ¿Por favor primero podríamos atender los comentarios de arriba?

devarops commented 2 years ago

Creo que ya quedó. Sólo esperaré a que termine el Actions. 😉

devarops commented 2 years ago

Este PR quedó de buen tamaño 👍🏾

image

Creo que ya quedó. ¿Por favor podrías presionar el botón Re-request review cuando termine de correr el Actions?

mvillasante commented 2 years ago

Está corriendo el Actions nuevamente... Todavía lo olvido... Pensé que los cambios que hice no iban a afectar el formato...

Acabo de notar que dice "Billable time". ¿Cobra github por esta característica?

Espero no deberles demasiado después de este examen... :see_no_evil:

devarops commented 2 years ago

Acabo de notar que dice "Billable time". ¿Cobra github por esta característica?

No te preocupes. Te lo vamos a descontar de tu primer cheque 😜

mvillasante commented 2 years ago

Acabo de notar que dice "Billable time". ¿Cobra github por esta característica?

No te preocupes. Te lo vamos a descontar de tu primer cheque stuck_out_tongue_winking_eye

Me conformo con que llegue el primero!