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

Split CI workflows into gfortran and containerized #88

Merged
merged 41 commits into from
Mar 16, 2024

Conversation

alejandrobodas
Copy link
Collaborator

This PR split the continuous integration tests into two workflows, one for the gfortran compilers and a second one for containerized ifort compilers. This PR also creates KGOs for ifort and ifx that allows for strict regression tests (as it was done for gfortran compilers).
This is an interim step that fixes the CI workflows. Support for nvidia compiler will be added in a separate PR because that compiler throws a runtime error that needs to be investigated.

@alejandrobodas alejandrobodas self-assigned this Mar 15, 2024
@alejandrobodas
Copy link
Collaborator Author

@RobertPincus would you mind reviewing this one? I'll tidy up the download of the KGOs in a separate pull request, but I believe it would be useful to get this onto the master branch to avoid delaying the other PRs that are open. Thanks!

Copy link
Contributor

@RobertPincus RobertPincus left a comment

Choose a reason for hiding this comment

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

Thanks a lot. Looks good, I will be happy to see more graceful downloading via some Github action

@alejandrobodas alejandrobodas merged commit 0aa5d45 into master Mar 16, 2024
10 checks passed
@alejandrobodas
Copy link
Collaborator Author

Thanks @RobertPincus . Changes merged.

@alejandrobodas alejandrobodas deleted the ci_workflow_test branch March 16, 2024 12:23
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.

2 participants