Skip to content

refactor(gui): eliminar lógica redundante en el manejo de operadores - #46

Merged
Jhos3ph merged 2 commits into
devfrom
hotfix/remove-redundant-code
Nov 27, 2025
Merged

refactor(gui): eliminar lógica redundante en el manejo de operadores#46
Jhos3ph merged 2 commits into
devfrom
hotfix/remove-redundant-code

Conversation

@Jhos3ph

@Jhos3ph Jhos3ph commented Nov 27, 2025

Copy link
Copy Markdown
Contributor

📋 Descripción

Se ha eliminado un bloque de código redundante en el método operation_click de gui.py. Este bloque elif intentaba manejar casos de operadores encadenados que ya son gestionados correctamente por la lógica principal, simplificando así el flujo de control y evitando posibles efectos secundarios no deseados.


🎯 Tipo de Cambio

  • 🐛 Bug fix (corrección de error)
  • ✨ Nueva funcionalidad
  • 📝 Mejora de documentación
  • ♻️ Refactoring (sin cambios de funcionalidad)
  • 🧪 Tests (agregar o mejorar tests)
  • 🎨 Estilo (formateo, nombres de variables)
  • 🚀 Release (preparación para release)

🧪 ¿Cómo se ha probado?

  • Tests unitarios
  • Prueba manual
  • Probado en diferentes sistemas operativos

@Jhos3ph Jhos3ph self-assigned this Nov 27, 2025
@Jhos3ph Jhos3ph added the mejora Mejora a funciones existentes label Nov 27, 2025

@jandrescodes jandrescodes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok, eliminemos este código redundante.

@alexricardotapiacarita-ai alexricardotapiacarita-ai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

esta bien

@Jhos3ph
Jhos3ph merged commit 468ec52 into dev Nov 27, 2025
2 checks passed
@jandrescodes
jandrescodes deleted the hotfix/remove-redundant-code branch November 30, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mejora Mejora a funciones existentes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants