Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@jmaxter
Copy link
Contributor

@jmaxter jmaxter commented Mar 9, 2023

Close #1970

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

¡Muchas gracias @jmaxter por tu primera contribución! Se ve muy bien en general 👍, aunque como te diste cuenta había un problema con el archivo de TRANSLATORS. Te dejo un comentario al respecto para que lo soluciones, además de otros detallitoss que haría falta arreglar antes de poder hacer un merge.

¡Mucho ánimo! 💪

TRANSLATORS Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
@jmaxter
Copy link
Contributor Author

jmaxter commented Mar 10, 2023

Enviado corrección de archivo TRANSLATORS y library/calendar.

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Gracias @jmaxter se ve mucho mejor ahora :) queda un par de detalles y creo que ya vamos a estar listos para hacer el merge :) 👍

library/calendar.po Outdated Show resolved Hide resolved
library/calendar.po Outdated Show resolved Hide resolved
@jmaxter
Copy link
Contributor Author

jmaxter commented Mar 11, 2023

Finalizado library/calendar

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Estamos listos, todos los tests pasan ahora, así que vamos a mergear! 🎉

Un detalle que olvide mencionar: edité la descripción del PR para que al hacer el merge se cierre automáticamente el issue asociado.

@rtobar rtobar merged commit d2741af into python:3.11 Mar 11, 2023
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.

Translate 'library/calendar.po'

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.