Skip to content

Also test with latest release#327

Open
masterleinad wants to merge 1 commit intokokkos:developfrom
masterleinad:test_with_latest_stable
Open

Also test with latest release#327
masterleinad wants to merge 1 commit intokokkos:developfrom
masterleinad:test_with_latest_stable

Conversation

@masterleinad
Copy link
Copy Markdown
Contributor

@masterleinad masterleinad commented Mar 11, 2026

Setting up for whatever range of versions we want to support in the end.

@masterleinad masterleinad force-pushed the test_with_latest_stable branch 2 times, most recently from 7ab57a8 to a81bfe4 Compare March 11, 2026 21:12
@masterleinad masterleinad force-pushed the test_with_latest_stable branch from 5d9589b to 1a6a8f7 Compare March 12, 2026 23:42
@masterleinad masterleinad marked this pull request as ready for review March 13, 2026 00:03
fail-fast: false
matrix:
include:
kokkos-version-tag: [5.0.2, develop]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we should test against develop in the CI but that belongs to nightlies or weeklies instead

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's pre-existing at least.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree, I would take our latest 5 and latest 4 since we kind of promised long term support

Copy link
Copy Markdown
Contributor

@vlkale vlkale left a comment

Choose a reason for hiding this comment

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

Do we also want to test for backward compatibility with earlier versions of Kokkos, say, Kokkos 4.5?

I agree at minimum a latest major release version of Kokkos (Kokkos 5.x right now) ought to be CI tested against Kokkos Tools weekly and even nightly.

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.

4 participants