Skip to content

fix(cudart-sys): strip the "V" from the version string#73

Merged
robik75 merged 1 commit intomainfrom
rr/cuda-version-fix
Feb 3, 2025
Merged

fix(cudart-sys): strip the "V" from the version string#73
robik75 merged 1 commit intomainfrom
rr/cuda-version-fix

Conversation

@robik75
Copy link
Member

@robik75 robik75 commented Feb 2, 2025

What ❔

This PR fixes a a bug in the CUDA version detection.

Why ❔

The returned version string fas in the format Vx.y.z instead of x.y.z.

@robik75 robik75 self-assigned this Feb 2, 2025
@robik75 robik75 requested a review from mcarilli February 2, 2025 16:23
@robik75 robik75 force-pushed the rr/cuda-version-fix branch from 5a7fa75 to 11198ed Compare February 3, 2025 16:32
@mcarilli mcarilli self-requested a review February 3, 2025 16:33
@robik75 robik75 merged commit 76cfc1c into main Feb 3, 2025
15 checks passed
@robik75 robik75 deleted the rr/cuda-version-fix branch February 3, 2025 16:43
Artemka374 pushed a commit that referenced this pull request Feb 7, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.152.12](v0.152.11...v0.152.12)
(2025-02-07)


### Features

* Bump crypto versions
([#76](#76))
([b8cec04](b8cec04))
* **cudart:** refactor allocations and stream
([#75](#75))
([9c0bedc](9c0bedc))


### Bug Fixes

* **ci:** bump deprecated actions/download-artifact and
actions/upload-artifact v3 to v4
([#74](#74))
([5727c7c](5727c7c))
* **ci:** Update github-hosted runner label
([#70](#70))
([822c276](822c276))
* **cudart-sys:** strip the "V" from the version string
([#73](#73))
([76cfc1c](76cfc1c))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: zksync-era-bot <zksync-era-bot@users.noreply.github.com>
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