-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[move-vm] Make bytecode version v7 the default #14876
base: main
Are you sure you want to change the base?
Conversation
⏱️ 2h 17m total CI duration on this PR
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #14876 +/- ##
===========================================
- Coverage 72.6% 59.8% -12.9%
===========================================
Files 2400 853 -1547
Lines 485208 208243 -276965
===========================================
- Hits 352656 124530 -228126
+ Misses 132552 83713 -48839 ☔ View full report in Codecov by Sentry. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
We could not do that before because mainnet did not support v7 yet, but that has changed now.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Forge suite
|
✅ Forge suite
|
✅ Forge suite
|
@@ -518,9 +518,9 @@ pub const VERSION_8: u32 = 8; | |||
pub const VERSION_MAX: u32 = VERSION_7; | |||
|
|||
/// Mark which version is the default version. This is the version used by default by tools like | |||
/// the compiler. Notice that this version might be different than the one supported on nodes. | |||
/// the compiler. Notice that this version might be different from the one supported on nodes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: "different than" is perfectly acceptable in American English, although it seems that "different from" is preferred in writing. Wasn't really worth changing, but not worth arguing about, either.
Brits randomly use "different to" which always sounds very wrong to me.
Description
We could not do that before because mainnet did not support v7 yet, but that has changed now.
How Has This Been Tested?
Existing tests
Type of Change
Which Components or Systems Does This Change Impact?