Open asanzo opened 2 years ago
@nscarcella ¿qué opinás? Es razonable esto? Quizás es atar wollok-language a wollok-ts más de lo que queremos.
Hola @asanzo ! la verdad es que sí, es medio una flojera mergear y no darte cuenta de que rompés algo, pero por el otro lado nos pasa que un cambio necesita a veces actualizar language, luego ts. Entonces si language está atado a ts y ts a language, entramos en una doble referencia de la que es difícil salir. @PalumboN @ivojawer qué opinan? Podríamos correr el CI sobre wollok-ts pero no frenar el PR... Abrazo Fer
Hoy muchas veces estamos mergeando PRs a language medio "en el aire". Un PR podría tranquilamente romper porque te olvidaste de correr los tests.
Propongo que esto se haga con wollok-ts, con la última versión de master.
Es decir, este issue debería:
npm install
pero asegurándose que el language que descargue en la carpetalanguage
sea pisado por el language actual (para probar los cambios actuales) y luego unnpm run test:sanity && npm run test:examples
, quizás.