onflow / atree

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

Update for Cadence integration for atree inlining and deduplication #352

Closed fxamacker closed 11 months ago

fxamacker commented 11 months ago

Closes #351 Updates #292 https://github.com/onflow/cadence/issues/2809 https://github.com/onflow/flow-go/issues/1744

This PR adds new interfaces, changes error types, exports new functions, etc. This is to resolve all the integration issues integration issues I ran into while updating onflow/cadence to use Atree PR #342 and #345.

Changes:


codecov-commenter commented 11 months ago

Codecov Report

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

Comparison is base (108dc31) 62.56% compared to head (eb9a503) 62.58%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## fxamacker/add-composite-type-to-smoke-test #352 +/- ## ============================================================================== + Coverage 62.56% 62.58% +0.01% ============================================================================== Files 15 15 Lines 10566 10602 +36 ============================================================================== + Hits 6611 6635 +24 - Misses 3010 3016 +6 - Partials 945 951 +6 ``` | [Files](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow) | Coverage Δ | | |---|---|---| | [storable.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-c3RvcmFibGUuZ28=) | `60.20% <100.00%> (+0.82%)` | :arrow_up: | | [storage.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-c3RvcmFnZS5nbw==) | `73.69% <100.00%> (+0.59%)` | :arrow_up: | | [typeinfo.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-dHlwZWluZm8uZ28=) | `60.32% <100.00%> (ø)` | | | [array.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-YXJyYXkuZ28=) | `68.08% <77.77%> (-0.03%)` | :arrow_down: | | [encode.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-ZW5jb2RlLmdv) | `76.47% <50.00%> (-4.26%)` | :arrow_down: | | [map.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-bWFwLmdv) | `65.28% <65.00%> (+0.05%)` | :arrow_up: | | [array\_debug.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-YXJyYXlfZGVidWcuZ28=) | `47.25% <27.77%> (-0.20%)` | :arrow_down: | | [map\_debug.go](https://app.codecov.io/gh/onflow/atree/pull/352?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=onflow#diff-bWFwX2RlYnVnLmdv) | `43.94% <50.79%> (+0.50%)` | :arrow_up: |

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