onflow / atree

Atree provides scalable arrays and scalable ordered maps.
https://onflow.org
Apache License 2.0
39 stars 13 forks source link

Add support for changing type info of atree arrays (atree inlining branch) #376

Closed fxamacker closed 3 months ago

fxamacker commented 4 months ago

Updates #372

This feature is needed by Cadence migrations to update static types. For more details, see

This PR adds Array.SetType() to allow updating TypeInfo.

NOTE: Implementation of this in atree inlining branch is different than non-inlining branch(es).


codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 58.64198% with 67 lines in your changes are missing coverage. Please review.

Project coverage is 63.22%. Comparing base (2fbf860) to head (8c7c31e).

Files Patch % Lines
array.go 62.96% 30 Missing :warning:
map.go 55.22% 30 Missing :warning:
storage.go 50.00% 3 Missing and 1 partial :warning:
storable_slab.go 0.00% 2 Missing :warning:
basicarray.go 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## feature/array-map-inlining #376 +/- ## ============================================================== + Coverage 62.45% 63.22% +0.76% ============================================================== Files 15 15 Lines 10919 10791 -128 ============================================================== + Hits 6820 6823 +3 + Misses 3119 3005 -114 + Partials 980 963 -17 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.