Skip to content

ci: add ci build#276

Merged
devcrocod merged 13 commits intodevelopfrom
ci/add-ci-check
Feb 28, 2026
Merged

ci: add ci build#276
devcrocod merged 13 commits intodevelopfrom
ci/add-ci-check

Conversation

@devcrocod
Copy link
Collaborator

@devcrocod devcrocod commented Feb 28, 2026

closes #244

Copilot AI review requested due to automatic review settings February 28, 2026 02:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions CI workflow to build the native OpenBLAS JNI library and run Multik’s JVM + Linux Native test suites on pushes/PRs targeting develop.

Changes:

  • Introduces .github/workflows/build.yml to run CI on push, pull_request, and manual dispatch.
  • Installs GCC/GFortran/CMake tooling and builds :multik-openblas:build_cmake.
  • Runs jvmTest and linuxX64Test, then uploads test reports as artifacts.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

GCC 15 is not available in Ubuntu 24.04 (ubuntu-latest) repositories,
causing the CI build to fail. GCC 14 is available out of the box.
Display test results directly in the job summary using
mikepenz/action-junit-report.
uses: actions/setup-java@v4
with:
distribution: temurin
java-version: '21'
Copy link

Choose a reason for hiding this comment

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

Is the minimum required JDK still JDK 8? If so, it’d make sense to use that for testing instead of JDK 21 to ensure compatibility.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good point
However, the purpose of this workflow is not to verify jdk compatibility, since the main builds on TC already use the minimum supported JDK version
This workflow is primarily intended to provide transparency and visibility for pull requests to contributors and users

@devcrocod devcrocod merged commit 47fe41a into develop Feb 28, 2026
2 checks passed
@devcrocod devcrocod deleted the ci/add-ci-check branch February 28, 2026 23:22
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.

Add CI check workflow for PRs

3 participants