Skip to content

[CI] Do not hard-code OCK version number#1002

Merged
hvdijk merged 1 commit into
uxlfoundation:mainfrom
coldav:colin/fix_remote_hal
Sep 23, 2025
Merged

[CI] Do not hard-code OCK version number#1002
hvdijk merged 1 commit into
uxlfoundation:mainfrom
coldav:colin/fix_remote_hal

Conversation

@coldav

@coldav coldav commented Sep 23, 2025

Copy link
Copy Markdown
Collaborator

Overview

[CI] Do not hard-code OCK version number.

Reason for change

As we are preparing for a 5.0 release, tests should not assume 4.0.

Description of change

Describe the intended behaviour your changes are meant to introduce to the
project and explain how they resolve the problem stated above. Detail any
relevant changes that may affect other users of the project, such as
compilation options, runtime flags, expected inputs and outputs, API entry
points, etc.

If you have added new testing, provide details on what tests you have added
and what the purpose of them is.

Anything else we should know?

If there's any other relevant information we should know that may help us in
understanding and verifying your patch, please include it here.

Checklist

  • Read and follow the project Code of Conduct.
  • Make sure the project builds successfully with your changes.
  • Run relevant testing locally to avoid regressions.
  • Run clang-format-20 on all modified code.

@coldav coldav requested a review from a team as a code owner September 23, 2025 08:55
@coldav coldav force-pushed the colin/fix_remote_hal branch 10 times, most recently from 65f0f5a to 926323e Compare September 23, 2025 11:57
As we are preparing for a 5.0 release, tests should not assume 4.0.
@hvdijk hvdijk force-pushed the colin/fix_remote_hal branch from 926323e to a663c59 Compare September 23, 2025 13:39
@hvdijk hvdijk changed the title DRAFT: debug remote hal [CI] Do not hard-code OCK version number Sep 23, 2025
@hvdijk hvdijk merged commit 961c1be into uxlfoundation:main Sep 23, 2025
22 checks passed
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