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

Si me olvido paréntesis de un método, el error es feo y está en cualquier lado #82

Open
asanzo opened this issue Jul 1, 2023 · 4 comments
Labels
component: linter Linter / Validator reporter severity: 3 - major High priority issue UX User eXperience enhancement
Milestone

Comments

@asanzo
Copy link
Contributor

asanzo commented Jul 1, 2023

Si me olvido paréntesis de un método, el error es feo y está en cualquier lado
Con este código:

object pepita {
    method vola = 1
}

El error es: "La siguiente regla falló: malformed entity"
image

Y en lugar de mostrarlo sobre el method lo muestra sobre la palabra "object". Estaría bueno que lo tire más cerca (x ejemplo en el nombre del método o en la palabra method).

@fdodino
Copy link
Contributor

fdodino commented Aug 6, 2023

@PalumboN me parece que el problema es del parser, hay que confirmarlo con @nscarcella pero apostaría 20 euros...

@nscarcella
Copy link
Member

Cuando se falla en parsear un miembro de un objeto el parser se recupera y agrega un error a la entidad padre. En este caso, el mètodo no está parseando, con lo cual el objeto pepita debe tener adentro un error que dice la causa y el rango de caracteres donde falló.

Pero no es el parser el que relaciona el error con la zona a subrallar, así que yo mirarìa si:
a) El error de parseo que queda en el nodo Singleton está bien asociado al rango de caracteres que no se pudo parsear (Yo creo que debe estar bien, porque eso está testeado pero, de no ser así habría un problema en el parser).
b) Estamos usando esa informaciòn para marcar el texto correcto en el IDE.

@PalumboN
Copy link
Contributor

PalumboN commented Aug 20, 2023

Después de uqbar-project/wollok-ts#162 este código rompe el parser.
Antes no lo hacía (se introdujo ahí para solucionar parte de este otro issue: #83).

Como consecuencia de eso, el IDE ahora marca todo el archivo en rojo:
image

Hace falta más laburo.

@PalumboN PalumboN added bug Something isn't working severity: 3 - major High priority issue UX User eXperience enhancement and removed bug Something isn't working labels Mar 10, 2024
@PalumboN
Copy link
Contributor

PalumboN commented Aug 6, 2024

El estado actual de esto es este:
image

Marca el error donde queremos 👍
El/Los mensaje(s) que tira sigue siendo feo 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: linter Linter / Validator reporter severity: 3 - major High priority issue UX User eXperience enhancement
Projects
None yet
Development

No branches or pull requests

4 participants