Skip to content

Conversation

@DaniPopes
Copy link
Contributor

Update thiserror to v2, which includes no_std support.

@JulianGCalderon
Copy link
Contributor

Hi @DaniPopes ! Thanks for the PR. Could you update the changelog?

You can use this PR as an example: https://github.com/lambdaclass/cairo-vm/pull/1789/files#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed

@DaniPopes
Copy link
Contributor Author

Done

@edg-l edg-l requested a review from noaov1 as a code owner March 10, 2025 10:26
@gabrielbosio gabrielbosio enabled auto-merge March 17, 2025 16:14
@gabrielbosio gabrielbosio added this pull request to the merge queue Mar 17, 2025
Merged via the queue into lambdaclass:main with commit f1909dd Mar 17, 2025
181 of 184 checks passed
@DaniPopes DaniPopes deleted the thiserror-no-std branch March 18, 2025 02:12
gabrielbosio added a commit that referenced this pull request May 21, 2025
* Replace thiserror-no-std with thiserror 2

* Fix fuzzer using old thiserror-no-std

---------

Co-authored-by: Edgar <[email protected]>
Co-authored-by: Edgar Luque <[email protected]>
Co-authored-by: Julian Gonzalez Calderon <[email protected]>
Co-authored-by: Gabriel Bosio <[email protected]>
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