Skip to content

Conversation

@mamunozcar
Copy link
Contributor

Version 2.14.0 🎉

Additions ➕

Changes ➖

None.

Fixes 🐛

None.

Copy link

@rudyjb24 rudyjb24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Se ve bien el código, dejo un comentario sobre la forma solamente. Es buena práctica hacer squash de los fixups antes de mandar a revisar, para facilitar la lectura al reviewer (si durante la revisión salen comentarios que arreglas con un fixup, esos me parece que esta bien dejarlos)

Lo otro, porque tienes merge commits de otros PR's en este PR?

@mamunozcar
Copy link
Contributor Author

Se ve bien el código, dejo un comentario sobre la forma solamente. Es buena práctica hacer squash de los fixups antes de mandar a revisar, para facilitar la lectura al reviewer (si durante la revisión salen comentarios que arreglas con un fixup, esos me parece que esta bien dejarlos)

Lo otro, porque tienes merge commits de otros PR's en este PR?

Lo que ocurre es que en este repositorio no tenemos configurado bors para que nos apoye con el despliegue a producción de los cambios. Esto hace que al final tenga que crearse un PR aparte (este) que se encargue de llevar los cambios a stable.

@mamunozcar mamunozcar requested a review from rudyjb24 November 11, 2025 16:29
@mamunozcar mamunozcar merged commit 4410a61 into stable Nov 11, 2025
6 checks passed
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.

3 participants