wollok icon indicating copy to clipboard operation
wollok copied to clipboard

Puedo importar el mismo archivo

Open PalumboN opened this issue 5 years ago • 2 comments

Desde example.wlk puedo importar los objetos de example.wlk image

Y eso ahora hace que salten los warnings de

There is already an imported reference with this name	example.wlk

:stuck_out_tongue:

PalumboN avatar Sep 14 '19 18:09 PalumboN

Ah, pillín!!!

Eso tiene dos aristas:

  1. Mostrar una advertencia si importás el archivo en el que estás parado
  2. Descartar la validación de "already imported reference" para imports sobre el mismo archivo

fdodino avatar Sep 16 '19 09:09 fdodino

Sobre 1 => Yo sería más estricto acá y marcaría error, en ningún caso tiene sentido autoimportarse, es obviamente un error. Sobre 2 => Más capo aún sería tocar el scope provider para ignorar el import malo. Pero bueno, es un toque más difícil.

De hecho, creo que si arrancás por el punto 2, rechazás el import y eso automáticamente pondría el error en 1. Habría que ver si el error queda lindo o no.

El lun., 16 de sep. de 2019 a la(s) 06:53, Fernando Dodino ( [email protected]) escribió:

Ah, pillín!!!

Eso tiene dos aristas:

  1. Mostrar una advertencia si importás el archivo en el que estás parado
  2. Descartar la validación de "already imported reference" para imports sobre el mismo archivo

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1795?email_source=notifications&email_token=ABDLKOM6QWF37H3PC4QIKJDQJ5JSDA5CNFSM4IWX3PC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6YUJ4A#issuecomment-531711216, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDLKOPUIF6SPQU6N44H363QJ5JSDANCNFSM4IWX3PCQ .

npasserini avatar Sep 16 '19 14:09 npasserini