Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reformat de clases/widgets complejas/os #9

Open
4 of 9 tasks
RafaelMoralesV opened this issue May 30, 2022 · 2 comments
Open
4 of 9 tasks

Reformat de clases/widgets complejas/os #9

RafaelMoralesV opened this issue May 30, 2022 · 2 comments
Assignees
Labels
❓ Question Further information is requested ♻️ Refactor Improvements of code

Comments

@RafaelMoralesV
Copy link
Member

RafaelMoralesV commented May 30, 2022

Description

Hay un par de archivos que contienen codigo extraño, complejo, demasiado largo, y, como en estos momentos donde no hay muchas mejoras pensadas para la app, puede ser el mejor momento para intentar arreglarlos.

Tengo un draft de PR (#8) donde estoy arreglando warnings. No estoy seguro del todo si debería ser en ese espacio donde hacer este reformat, o si crear una branch nueva; planteo esto por un problema de merging que se pueda generar a futuro. De todas formas, creo que la gran mayoria de estos archivos son antiguos y que no se estan tocando mucho.

Tipo de cambio

  • Documentación.
  • Mejora de codigo existente (performance y/o estilo del codigo y/o otra mejora)

Resources

En primera medida, estos son los archivos que me preocupan:

Archivos complejos o largos

  • screens/asignaturas_nota_tab.dart
  • screens/login_screen.dart
  • widgets/acerca_dialog.dart
  • widgets/acerca_screen.dart

Archivos con codigo comentado

  • screens/asignaturas_resumen_tab.dart *
  • screens/avancemalla_screen.dart *
  • screens/boletin_screen.dart *
  • widgets/bloque_ramo_card.dart *
  • widgets/carrera_list_item.dart *

Cualquier archivo que no se haya mencionado, que no sepan si se esta utilizando, tambien vale la pena mencionarlo en la discusión. Lo ideal es limpiar lo más posible ahora, cosa de no tener que hacerlo dos veces.

@RafaelMoralesV RafaelMoralesV self-assigned this May 30, 2022
@mapacheverdugo mapacheverdugo added ❓ Question Further information is requested ♻️ Refactor Improvements of code labels Jan 27, 2023
@Im-Fran
Copy link
Member

Im-Fran commented Nov 2, 2023

Voy a estar aplicando estos arreglos en una branch que hice en mi repo :p

@Im-Fran Im-Fran self-assigned this Nov 2, 2023
@Im-Fran
Copy link
Member

Im-Fran commented Jun 30, 2024

Esto se "avanzó" en #48, pero lo mantendré abierto hasta que tengamos completamente implementado clean architecture.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❓ Question Further information is requested ♻️ Refactor Improvements of code
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants