Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: migrate to go-libstoracha #132

Merged
merged 5 commits into from
Feb 17, 2025

Conversation

frrist
Copy link
Member

@frrist frrist commented Feb 14, 2025

@frrist frrist self-assigned this Feb 14, 2025
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/construct/construct.go 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
cmd/lambda/notifier/main.go 0.00% <ø> (ø)
cmd/lambda/remotesync/main.go 0.00% <ø> (ø)
cmd/query.go 0.00% <ø> (ø)
pkg/aws/bucketfallbackmapper.go 70.58% <ø> (ø)
pkg/aws/dynamoprovidercontexttable.go 0.00% <ø> (ø)
pkg/aws/s3store.go 0.00% <ø> (ø)
pkg/aws/service.go 0.00% <ø> (ø)
pkg/client/client.go 36.47% <ø> (ø)
pkg/internal/testutil/gen.go 83.82% <ø> (ø)
pkg/service/blobindexlookup/cachinglookup.go 100.00% <ø> (ø)
... and 10 more

... and 1 file with indirect coverage changes

@frrist frrist marked this pull request as ready for review February 17, 2025 19:29
@frrist frrist requested review from volmedo and a team February 17, 2025 19:40
Copy link
Member

@volmedo volmedo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@frrist frrist merged commit 4c1a1d3 into main Feb 17, 2025
8 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to go-libstoracha package
2 participants