Skip to content

Conversation

@laraPPr
Copy link
Collaborator

@laraPPr laraPPr commented Sep 12, 2025

No description provided.

bedroge
bedroge previously approved these changes Sep 12, 2025
Copy link
Collaborator

@bedroge bedroge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it locally and it worked, though we need to rerun the workflow for updating the json file too.

@bedroge
Copy link
Collaborator

bedroge commented Sep 12, 2025

@laraPPr do you know why the CI is failing?

@boegel
Copy link
Contributor

boegel commented Sep 12, 2025

@laraPPr do you know why the CI is failing?

Could it be because the JSON files are not updated yet? Do we need to do this in 2 steps?

@laraPPr
Copy link
Collaborator Author

laraPPr commented Sep 12, 2025

No @bedroge found it but I have ran into this before. I think this was a change in one of the dependencies. The Tests kept failing months ago when I was testing it so than I had to add those spaces. This now seems to be resolved so now they can be removed again

@bedroge bedroge merged commit 52389e3 into EESSI:main Sep 12, 2025
3 checks passed
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

Successfully merging this pull request may close these issues.

3 participants