Update error handling of actions

timfelle commented 3 weeks ago

This pull request updates the error handling of actions to improve the overall reliability and stability of the code. The changes include better error messages and improved error handling logic.

Please do not manually merge this PR. Update the following repository level changes on github:

Branch requirements should be assigned to the completion step for both main and develop.
- The completion job should be the only requirement set develop / Develop PR Ready since that job checks all the others. This allow us to extend the CI pipeline later, without modifying the branch protection rules.

EDIT: The previously mentioned repository variables does not support PR's submitted from forks. Those should be removed again.

Depends on #1303

timfelle commented 3 weeks ago

I have added the don't merge label so we can make sure it actually works before you merge it in.

timfelle commented 3 weeks ago

@njansson Alright, we are down to just the macos issues. I don't see what would cause the tests to fail. Do we run with a different version of pfunit?

I have removed the repository variables again. They do not propagate to pull requests opened from a fork, which make them kinda useless in this case. Instead i managed to use a top-level environment block in the workflow which should also make it simple to update if needed.

timfelle commented 3 weeks ago

MacOS issues are now fixed as per #1303

ExtremeFLOW / neko

Update error handling of actions #1301