application-research / autoretrieve

A server to make GraphSync data accessible on IPFS
22 stars 7 forks source link

deps: update to dag-pb version which allocates new slices #154

Closed rvagg closed 1 year ago

rvagg commented 1 year ago

Ref: https://github.com/ipld/go-codec-dagpb/pull/57 Ref: https://github.com/application-research/autoretrieve/issues/142#issuecomment-1328703179

150 didn't seem to have the impact I hoped, but I want to try this to see if it does--if this doesn't have an impact then I'm not looking in the right place.

Note this is an untagged (and unmerged) version of go-codec-dagpb; I'll revert this if it has no impact and if it does then look to merging and tagging that version (although there's some discussion to be had there about whether it's the right place to do this).

codecov-commenter commented 1 year ago

Codecov Report

Base: 20.84% // Head: 20.84% // No change to project coverage :thumbsup:

Coverage data is based on head (ace2df4) compared to base (861e2ce). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #154 +/- ## ======================================= Coverage 20.84% 20.84% ======================================= Files 16 16 Lines 2341 2341 ======================================= Hits 488 488 Misses 1823 1823 Partials 30 30 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.