-
Notifications
You must be signed in to change notification settings - Fork 137
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
Bump atree version in Cadence 0.42 for new features (e.g. deduplication of Cadence dictionary type info) #3177
Conversation
This commit bumps atree version to benefit from deduplication of atree inlined Cadence dictionary type info. For more info, see onflow/atree#369
Cadence Benchstat comparisonThis branch with compared with the base branch onflow:feature/atree-register-inlining-v0.42 commit 66e6d4f Collapsed results for better readability
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feature/atree-register-inlining-v0.42 #3177 +/- ##
======================================================================
Coverage 79.54% 79.55%
======================================================================
Files 336 336
Lines 79560 79560
======================================================================
+ Hits 63289 63293 +4
+ Misses 13958 13954 -4
Partials 2313 2313
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
d9cfa83
into
feature/atree-register-inlining-v0.42
This PR reduces memory and storage used by EN and also provides features useful for Cadence migrations.
More specifically, this bumps to newer atree version with improvements:
For more info, see onflow/atree PRs:
etc.
master
branchFiles changed
in the Github PR explorer