Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

(feat): Add support for Cloudflare KV storage and improve documentation #287

Merged
merged 20 commits into from
Feb 13, 2024

Conversation

AdiRishi
Copy link
Owner

@AdiRishi AdiRishi commented Feb 13, 2024

Description

Resolves #159 #248

This PR is a major refactor of the codebase

  • All storage operations are now done behind a generic StorageInterface
  • The new storage interface is able to abstract both KV and R2, allowing for both backends to be used
  • Major docs refresh, outlining both how to setup the project in a clearer way along with detailed guides on R2 and KV

How Has This Been Tested?

  • This worker will be deployed under a "next" variant and will be used in production for a bit before the PR is merged

Copy link

changeset-bot bot commented Feb 13, 2024

🦋 Changeset detected

Latest commit: 572b52d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
turborepo-remote-cache-cf Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

🌩 Deploying with Cloudflare Pages

Name Result
Last commit: 572b52d
Status: ✅ Deploy successful!
Preview URL: https://a386edc0.cloudflare-turbo-cache-docs.pages.dev

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7883229427

Details

  • -2 of 288 (99.31%) changed or added relevant lines in 10 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+5.3%) to 98.185%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/storage/r2-storage.ts 76 78 97.44%
Totals Coverage Status
Change from base Build 7881285713: 5.3%
Covered Lines: 506
Relevant Lines: 508

💛 - Coveralls

@AdiRishi AdiRishi merged commit 7851729 into master Feb 13, 2024
5 checks passed
@AdiRishi AdiRishi deleted the turbo-next branch February 13, 2024 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants