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(deno/idb): implement IDBObjectStore.get #41

Merged
merged 3 commits into from
Apr 22, 2024
Merged

feat(deno/idb): implement IDBObjectStore.get #41

merged 3 commits into from
Apr 22, 2024

Conversation

hasundue
Copy link
Owner

  • fix(deno/idb): abandon ambient type declaration
  • feat(deno/idb): implement IDBObjectStore.get
  • refactor(deno/idb): do not replace self.indexedDB

@hasundue hasundue changed the title app cache feat(deno/idb): implement IDBObjectStore.get Apr 22, 2024
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

Attention: Patch coverage is 72.54902% with 28 lines in your changes are missing coverage. Please review.

Project coverage is 84.49%. Comparing base (012f70c) to head (7495403).
Report is 10 commits behind head on main.

Files Patch % Lines
deno/idb/mod.ts 58.33% 19 Missing and 1 partial ⚠️
std/stores.ts 53.84% 6 Missing ⚠️
deno/idb/stores.ts 84.61% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   86.75%   84.49%   -2.27%     
==========================================
  Files          22       33      +11     
  Lines         725     1090     +365     
  Branches       52       88      +36     
==========================================
+ Hits          629      921     +292     
- Misses         95      166      +71     
- Partials        1        3       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hasundue hasundue merged commit a1d89a8 into main Apr 22, 2024
4 of 6 checks passed
@hasundue hasundue deleted the app-cache branch April 22, 2024 07:33
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.

1 participant