opennextjs / opennextjs-aws

Open-source Next.js adapter for AWS
https://opennext.js.org
MIT License
4.38k stars 130 forks source link

fix(cache): make dummy cache error ignorable #641

Open vicb opened 1 day ago

vicb commented 1 day ago

All "dummy" cache methods throw by design. This PR makes the corresponding errors ignorable so that the logs are less verbose and always use the DEBUG level.

changeset-bot[bot] commented 1 day ago

🦋 Changeset detected

Latest commit: 920a7fdeeeb6e5230467d181b554f7fb46c7f808

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

This PR includes changesets to release 3 packages | Name | Type | | ---------------- | ----- | | @opennextjs/aws | Patch | | app-pages-router | Patch | | app-router | Patch |

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

github-actions[bot] commented 1 day ago

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 22.33% 1676 / 7503
🔵 Statements 22.33% 1676 / 7503
🔵 Functions 55.05% 98 / 178
🔵 Branches 69.41% 413 / 595
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/open-next/src/adapters/cache.ts 77.13% 80.89% 85.71% 77.13% 112-113, 174-176, 179-187, 197, 242-253, 280, 409-418, 422, 433-478
packages/open-next/src/overrides/incrementalCache/dummy.ts 0% 0% 0% 0% 1-17
packages/open-next/src/utils/error.ts 25% 20% 25% 25% 13-20, 32-39, 43-50
Generated in workflow #774 for commit 920a7fd by the Vitest Coverage Report Action
pkg-pr-new[bot] commented 1 day ago

Open in Stackblitz

pnpm add https://pkg.pr.new/@opennextjs/aws@641

commit: 920a7fd