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

¿Agregar tests al CI de language? #144

Open
2 tasks
asanzo opened this issue Jul 18, 2022 · 2 comments
Open
2 tasks

¿Agregar tests al CI de language? #144

asanzo opened this issue Jul 18, 2022 · 2 comments

Comments

@asanzo
Copy link
Contributor

asanzo commented Jul 18, 2022

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:

  • Agregar una carpeta .github con un workflow de tests al language.
  • Que eso se clone el wollok-ts, y le corra un npm install pero asegurándose que el language que descargue en la carpeta language sea pisado por el language actual (para probar los cambios actuales) y luego un npm run test:sanity && npm run test:examples, quizás.
@asanzo
Copy link
Contributor Author

asanzo commented Jul 18, 2022

@nscarcella ¿qué opinás? Es razonable esto?
Quizás es atar wollok-language a wollok-ts más de lo que queremos.

@fdodino
Copy link
Contributor

fdodino commented May 24, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants