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

Add CI for the examples #48

Open
mdiazmel opened this issue Nov 6, 2020 · 2 comments
Open

Add CI for the examples #48

mdiazmel opened this issue Nov 6, 2020 · 2 comments

Comments

@mdiazmel
Copy link
Contributor

mdiazmel commented Nov 6, 2020

It should be nice to perform some kind of automated testing on the examples.

@lesteve
Copy link
Member

lesteve commented Nov 9, 2020

Yeah I agree this would be ideal but I don't know how practical this would be, unless the discussion about CI on Jean Zay actually turns into something real (probably not in the near future though).

For now, I think I am fine with having examples that we fix when people report problems.

@mdiazmel
Copy link
Contributor Author

mdiazmel commented Nov 9, 2020

The idea is to have a nice green marker when somebody submit a new PR, mainly when adding new examples, in order to test that nothing is broken and that the new code works well (i.e., that the slurm job van be submitted to Jean Zay).
I put it here to force myself to do some experiments and, eventually, to discuss about it with the JZ's admins ;)

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