Skip to content

Upgrade ruby to 3.4.8, rails to 7.2.3 and all linked gems #510

Open
jlhiguitac wants to merge 38 commits intojorge_higuita/LIB-354from
jorge_higuita/LIB-458
Open

Upgrade ruby to 3.4.8, rails to 7.2.3 and all linked gems #510
jlhiguitac wants to merge 38 commits intojorge_higuita/LIB-354from
jorge_higuita/LIB-458

Conversation

@jlhiguitac
Copy link
Contributor

🛠️ Changes

Upgrade all the gems
Upgrade ruby to 3.4.8
Upgrade rails to 7.2.3

📝 Associated issues

LIB-458

🤔 Considerations

These upgrades were necessary because there were security wholes, and ruby 3.1 is out of support.

✅ Checks

Update ruby local version to test these changes.

@jlhiguitac jlhiguitac self-assigned this Feb 4, 2026
Copy link
Member

@erikasv erikasv left a comment

Choose a reason for hiding this comment

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

Son preguntas, pero dependiendo de las respuestas puede que se requieran cambios


# Don't care if the mailer can't send.
config.action_mailer.raise_delivery_errors = false
config.action_mailer.perform_deliveries = false
Copy link
Member

Choose a reason for hiding this comment

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

esta opción la quitaron? Sé que más abajo estás poniendo opciones que básicamente lo deshabilitan, pero si sigue estando presente, me parece mejor dejarla

Copy link
Member

Choose a reason for hiding this comment

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

para qué es esto? ahora requiere de un redis?

Copy link
Member

Choose a reason for hiding this comment

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

por qué eliminaste estas propiedades?

Copy link
Member

Choose a reason for hiding this comment

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

misma pregunta del archivo en inglés, si es por los cambios en la tarea 354, deberían aparecer los reemplazos, o simplemente se desaparecieron esos cambios?

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

Comments