Skip to content

feat: allow pull to be optional in SyncManager #1638

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jpoehnelt
Copy link

No description provided.

Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d13ebf8) to head (5735eb0).

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1638   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           56        56           
  Lines         1798      1806    +8     
  Branches       417       422    +5     
=========================================
+ Hits          1798      1806    +8     

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

Copy link

codecov bot commented Apr 14, 2025

Bundle Report

Changes will increase total bundle size by 2.11kB (0.03%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@signaldb/sync* 151.18kB 2.11kB (1.42%) ⬆️

ℹ️ *Bundle size includes cached data from a previous commit

Affected Assets, Files, and Routes:

view changes for bundle: @signaldb/sync

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.mjs.map 867 bytes 53.98kB 1.63%
index.umd.js.map 842 bytes 51.56kB 1.66%
index.mjs 248 bytes 20.09kB 1.25%
index.umd.js 142 bytes 11.79kB 1.22%
SyncManager.d.ts 1 bytes 8.07kB 0.01%
sync.d.ts 14 bytes 1.56kB 0.91%

@jpoehnelt jpoehnelt force-pushed the main branch 2 times, most recently from 7930b9f to 5735eb0 Compare April 15, 2025 00:48
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