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

Mejorar el manejo de errores #49

Open
JuanFdS opened this issue Apr 2, 2021 · 1 comment
Open

Mejorar el manejo de errores #49

JuanFdS opened this issue Apr 2, 2021 · 1 comment

Comments

@JuanFdS
Copy link
Collaborator

JuanFdS commented Apr 2, 2021

Hay un par de problemas que noto con como estamos manejando errores en el type system (y va, en el codigo en general creo):

  • Hay lugares donde estamos retornando algo que o tiene un error o no ({ block: ... } o { error: ... }) y hay otros lugares donde tiramos excepciones. Y al menos yo me perdí con donde estamos haciendo cada cosa, así que habría que ver si tiene sentido mantener ambas formas o hacer siempre lo mismo, y tratar de que sea mas clara en la interfaz de las funciones cual hace cual si mantenemos ambas formas.
  • Donde tiramos excepciones estamos tirando siempre Error, creo que hay lugares donde tiene sentido que tengamos subclases de errores como TypeError para todo lo que sea del type system (así si hay algun otro error inesperado podemos mostrarlo como "Sucedio un error, ¿queres reportarlo?" en vez de mostrarlo directamente) y tambien podría ser algo como EvaluationError para los errores al reducir bloques.
@JuanFdS JuanFdS changed the title Mejorar el manejo de errores del Type System Mejorar el manejo de errores Apr 2, 2021
@PalumboN
Copy link
Collaborator

PalumboN commented Apr 11, 2021

+:100: a lanzar excepciones ante errores y a la clasificación que tiraste (junto con la idea de reportar errores: sería a través de los issues del repo, no?)

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

No branches or pull requests

2 participants