Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: upgrade clarinet and clarity dependencies #463

Closed
wants to merge 6 commits into from

Conversation

hugocaillard
Copy link
Contributor

@hugocaillard hugocaillard commented Dec 1, 2023

Description

clarity-vm = "=2.1.1" was unused, and because it's strict, it prevented the upgrade of clarity-vm on clarinet

wasmtime which is a new dependency in clarity-vm requires rust v1.71.0 or higher. So I set it in the dockerfile to 1.73.0 as in the rust-toolchain file

Checklist

  • All tests pass
  • / Tests added in this PR (if applicable)

@hugocaillard hugocaillard force-pushed the chore/remove-clarity-vm branch from 134dff7 to b0ec5b8 Compare December 1, 2023 14:04
@hugocaillard hugocaillard force-pushed the chore/remove-clarity-vm branch from b0ec5b8 to e76e66c Compare December 1, 2023 14:05
@hugocaillard hugocaillard requested a review from lgalabru December 1, 2023 14:22
@hugocaillard hugocaillard force-pushed the chore/remove-clarity-vm branch from a4d33ec to bdf3ae1 Compare December 1, 2023 16:18
@hugocaillard hugocaillard marked this pull request as draft December 1, 2023 16:48
@hugocaillard
Copy link
Contributor Author

Kind of ready of review, but maybe not ready to be merged right now. I'd like to make sure that the clarity-vm branch is and remains stable

@hugocaillard hugocaillard marked this pull request as ready for review December 8, 2023 18:26
@hugocaillard hugocaillard force-pushed the chore/remove-clarity-vm branch from 7dd9983 to a938e61 Compare December 21, 2023 14:34
@hugocaillard hugocaillard force-pushed the chore/remove-clarity-vm branch from a938e61 to 84455e6 Compare December 21, 2023 14:51
@hugocaillard
Copy link
Contributor Author

hugocaillard commented Jan 12, 2024

Supersed by #475

@hugocaillard hugocaillard deleted the chore/remove-clarity-vm branch March 28, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants