Skip to content

Revert "feat: support byte ranges"#353

Merged
kylehuntsman merged 15 commits intomainfrom
revert-306-rvagg/byte-range
Jul 12, 2023
Merged

Revert "feat: support byte ranges"#353
kylehuntsman merged 15 commits intomainfrom
revert-306-rvagg/byte-range

Conversation

@hannahhoward
Copy link
Collaborator

Reverts #306

@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2023

Codecov Report

Merging #353 (a6389dc) into main (6c8a3dd) will increase coverage by 0.33%.
The diff coverage is 71.42%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #353      +/-   ##
==========================================
+ Coverage   75.89%   76.22%   +0.33%     
==========================================
  Files          86       85       -1     
  Lines        6575     6394     -181     
==========================================
- Hits         4990     4874     -116     
+ Misses       1302     1247      -55     
+ Partials      283      273      -10     
Impacted Files Coverage Δ
cmd/lassie/fetch.go 32.71% <0.00%> (+0.14%) ⬆️
pkg/internal/itest/testpeer/peerhttpserver.go 80.00% <ø> (+11.06%) ⬆️
pkg/server/http/util.go 50.54% <55.55%> (-1.63%) ⬇️
pkg/internal/itest/testpeer/generator.go 71.33% <67.67%> (-2.58%) ⬇️
pkg/verifiedcar/verifiedcar.go 83.22% <72.72%> (+2.60%) ⬆️
pkg/internal/testutil/toblocks.go 76.31% <100.00%> (+0.64%) ⬆️
pkg/retriever/bitswapretriever.go 93.88% <100.00%> (+0.05%) ⬆️
pkg/server/http/ipfs.go 65.96% <100.00%> (-1.33%) ⬇️
pkg/storage/duplicateaddercar.go 91.81% <100.00%> (-0.15%) ⬇️
pkg/types/request.go 76.92% <100.00%> (-5.99%) ⬇️
... and 1 more

... and 6 files with indirect coverage changes

@kylehuntsman kylehuntsman merged commit 631763e into main Jul 12, 2023
@kylehuntsman kylehuntsman deleted the revert-306-rvagg/byte-range branch July 12, 2023 23:53
kylehuntsman added a commit that referenced this pull request Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants