Skip to content

allow long client version in service version check'#61

Merged
raydouglass merged 1 commit intobranch-25.05from
clientcheck
Jun 2, 2025
Merged

allow long client version in service version check'#61
raydouglass merged 1 commit intobranch-25.05from
clientcheck

Conversation

@tmckayus
Copy link
Copy Markdown
Contributor

@tmckayus tmckayus commented Jun 2, 2025

Only consider the first two fields in the client version. Without this change the service will report that the version is a bad format, which is not necessary in this case.

@tmckayus tmckayus self-assigned this Jun 2, 2025
@tmckayus tmckayus requested a review from a team as a code owner June 2, 2025 15:51
@tmckayus tmckayus requested a review from rgsl888prabhu June 2, 2025 15:51
@tmckayus tmckayus added the bug Something isn't working label Jun 2, 2025
@tmckayus tmckayus requested review from Iroy30 and rgsl888prabhu and removed request for rgsl888prabhu June 2, 2025 15:51
@rgsl888prabhu rgsl888prabhu added the non-breaking Introduces a non-breaking change label Jun 2, 2025
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/merge

@rgsl888prabhu rgsl888prabhu removed the request for review from Iroy30 June 2, 2025 17:08
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/merge

@raydouglass raydouglass merged commit 5314217 into branch-25.05 Jun 2, 2025
58 checks passed
@rgsl888prabhu rgsl888prabhu deleted the clientcheck branch October 29, 2025 16:26
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Only consider the first two fields in the client version. Without this
change the service will report that the version is a bad format, which
is not necessary in this case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants