Skip to content

Update intel versions in CI#567

Merged
jatkinson1000 merged 1 commit intomainfrom
intel-bump
Mar 24, 2026
Merged

Update intel versions in CI#567
jatkinson1000 merged 1 commit intomainfrom
intel-bump

Conversation

@jatkinson1000
Copy link
Copy Markdown
Member

  • Resolves issue where tests were failing for specific versions of ifx in other branches.
  • Requires separate versions for ifort and ifx as ifort deprecated from 2024.2.0
  • Fixes a small bug in specifying the standard via command-line flag.

Comment thread .github/workflows/test_suite_ubuntu_cpu_intel.yml Fixed
* Resolves issue where tests were failing for specific versions of ifx
  in other branches.
* Requires separate versions for ifort and ifx as ifort deprecated
  from 2024.2.0
* Fixes a small bug in specifying the standard via command-line flag.
Copy link
Copy Markdown
Collaborator

@joewallwork joewallwork left a comment

Choose a reason for hiding this comment

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

One minor Zizmor query but otherwise looks good.


- name: Install Intel tools
uses: rscohn2/setup-oneapi@2ad0cf6b74bc2426bdcee825cf88f9db719dd727 # v0.1.0
uses: rscohn2/setup-oneapi@a5ae4e7697b9cd5dc8a616f94c1e56b87140999d
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Include tag comment? Or does it not correspond to one?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Didn't correspond to tag unfortunately, last tag available is the v0.1.0 that we used.
I'll perhaps open an issue on the repo to request a new release.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@jatkinson1000 jatkinson1000 merged commit 6e043d7 into main Mar 24, 2026
5 checks passed
@jatkinson1000 jatkinson1000 deleted the intel-bump branch March 24, 2026 07:58
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