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.

Sign up for GitHub

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

Jump to bottom

chore: upgrade clarity-vm including epoch 3.0 #1266

Merged
merged 47 commits into from
Jan 23, 2024
Merged

chore: upgrade clarity-vm including epoch 3.0 #1266

merged 47 commits into from
Jan 23, 2024

Conversation

hugocaillard
Copy link
Collaborator

@hugocaillard hugocaillard commented Nov 29, 2023 •
edited
Loading

Description

  • the clarity-vm dependency is now loaded from github instead of crates.io, it will remain the case while this is in active development
  • handle epochs 2.5 and 3.0
  • update some imports, especially from stacks-common
  • new codec wire format

This PR also upgrades some important dependencies:

  • tokio
  • reqwest
  • bollard
  • tui -> ratatui

@hugocaillard hugocaillard marked this pull request as draft November 29, 2023 18:08
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch 6 times, most recently from 4393a3b to aa9c5a8 Compare December 5, 2023 15:00
@codecov Codecov
Copy link

codecov bot commented Dec 5, 2023 •
edited
Loading

Codecov Report

Attention: 649 lines in your changes are missing coverage. Please review.

Comparison is base (dde449a) 41.82% compared to head (03d70b2) 41.79%.

Files Patch % Lines
components/clarity-repl/src/repl/interpreter.rs 64.78% 131 Missing ⚠️
...omponents/stacks-network/src/chains_coordinator.rs 0.00% 111 Missing ⚠️
components/stacks-network/src/ui/ui.rs 0.00% 82 Missing ⚠️
components/clarity-repl/src/codec/mod.rs 0.00% 79 Missing ⚠️
components/stacks-network/src/orchestrator.rs 0.00% 56 Missing ⚠️
components/stacks-network/src/ui/mod.rs 0.00% 52 Missing ⚠️
components/clarinet-files/src/network_manifest.rs 0.00% 23 Missing ⚠️
components/stacks-network/src/ui/app.rs 0.00% 21 Missing ⚠️
components/clarinet-files/src/project_manifest.rs 5.88% 16 Missing ⚠️
components/clarity-repl/src/repl/session.rs 40.00% 12 Missing ⚠️
... and 12 more
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #1266      +/-   ##
===========================================
- Coverage    41.82%   41.79%   -0.03%     
===========================================
  Files           87       87              
  Lines        31381    32028     +647     
===========================================
+ Hits         13124    13387     +263     
- Misses       18257    18641     +384

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hugocaillard hugocaillard requested a review from obycode December 5, 2023 17:53
@hugocaillard hugocaillard marked this pull request as ready for review December 5, 2023 17:58
obycode
obycode previously approved these changes Dec 5, 2023
View reviewed changes
Copy link
Member

@obycode obycode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm! I just had one minor question.

@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from c42d336 to 1074bd4 Compare December 6, 2023 15:12
obycode
obycode approved these changes Dec 6, 2023
View reviewed changes
Copy link
Member

@obycode obycode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

obycode
obycode previously approved these changes Dec 6, 2023
View reviewed changes
Copy link
Member

@obycode obycode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌

lgalabru
lgalabru previously approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@lgalabru lgalabru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exciting, thanks @hugocaillard!

csgui
csgui previously approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@csgui csgui left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done! LGTM!

@lgalabru lgalabru dismissed stale reviews from csgui, obycode, and themself via 6faae00 December 7, 2023 21:49
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from 6faae00 to c782152 Compare December 8, 2023 18:33
@hugocaillard hugocaillard mentioned this pull request Dec 11, 2023
Closed
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from 3f8b0da to cdf408c Compare December 12, 2023 15:04
@hugocaillard hugocaillard mentioned this pull request Dec 12, 2023
Closed
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from 4a11d5c to dca5567 Compare December 13, 2023 16:40
@hugocaillard hugocaillard marked this pull request as draft December 13, 2023 16:41
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from dca5567 to d3b0927 Compare December 13, 2023 17:25
MicaiahReid
MicaiahReid suggested changes Jan 17, 2024
View reviewed changes
hugocaillard and others added 2 commits January 17, 2024 17:55
@hugocaillard @MicaiahReid
fix: expose NAKA_ images
909610d 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
@hugocaillard @MicaiahReid
fix: typo
248562a 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
hugocaillard and others added 5 commits January 17, 2024 19:50
@hugocaillard @MicaiahReid
refactor: remove comment
8d70d71 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
@hugocaillard @MicaiahReid
fix: typo
de40306 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
@hugocaillard @MicaiahReid
fix: typo
e5e6003 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
@hugocaillard
fix: stacks-node config
0f34af6
@hugocaillard
fix: send stacking order at cycle position 1
9f39bcd
@hugocaillard hugocaillard mentioned this pull request Jan 19, 2024
Closed
3 tasks
@hugocaillard @MicaiahReid
fix: typo
c169884 
Co-authored-by: Micaiah Reid <micaiahreid@gmail.com>
MicaiahReid
MicaiahReid previously approved these changes Jan 19, 2024
View reviewed changes
Copy link
Contributor

@MicaiahReid MicaiahReid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have one optional note to add a comment, but I think it's good to merge!

@hugocaillard hugocaillard mentioned this pull request Jan 22, 2024
Closed
@hugocaillard hugocaillard force-pushed the chore/upgrade-clarity-vm branch from fd6f27e to be2d7e8 Compare January 22, 2024 16:17
@hugocaillard hugocaillard merged commit e3ce0c2 into develop Jan 23, 2024
17 checks passed
@hugocaillard hugocaillard deleted the chore/upgrade-clarity-vm branch January 23, 2024 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MicaiahReid MicaiahReid MicaiahReid approved these changes

@csgui csgui csgui left review comments

@lgalabru lgalabru lgalabru left review comments

@obycode obycode obycode left review comments

Assignees
No one assigned
Labels
None yet
Projects
DevTools
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@hugocaillard @csgui @lgalabru @obycode @MicaiahReid