Refactor the CI configuration to use inclusion instead of exclusion#3616
Refactor the CI configuration to use inclusion instead of exclusion#3616boegel merged 4 commits intoeasybuilders:developfrom
Conversation
This provides more clarity on what exactly is run
boegel
left a comment
There was a problem hiding this comment.
@Flamefire lgtm, that looks a bit cleaner indeed.
Let's bump to the latest Lmod 8.x while we're at it: Flamefire#7
|
I'd like to try something more: Avoid the repetition of the lmod version. But what we just need here is something we can access with |
OK! :) |
|
@Flamefire Once you're happy with the changes, I'll need to re-configure the test configurations that are required to pass (which is why there's a bunch of orange dots sitting there "waiting" right now). |
b2ae014 to
69ce2f7
Compare
|
I think this works now. What do you think about this approach? That's a bit shorter but sets those as actual env variables, so my current approach feels "more right" |
This provides more clarity on what exactly is run
The jobs have not changed, the following configurations are tested before (and after):