-
Notifications
You must be signed in to change notification settings - Fork 60
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
change default features to be none #339
change default features to be none #339
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #339 +/- ##
==========================================
- Coverage 74.89% 74.87% -0.03%
==========================================
Files 43 43
Lines 8409 8409
Branches 8409 8409
==========================================
- Hits 6298 6296 -2
- Misses 1724 1726 +2
Partials 387 387 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we also update the readme to reflect that users have to opt into a client via the features flag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with this as long as we update the README as steve requested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the doc updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you mention that the sync engine only supports local storage
6ab16ec
to
783199c
Compare
remove sync-engine from default features so there are no features enabled by default.
This reverts commit 57bf817.
remove sync-engine from default features so there are no features enabled by default. note this will required delta-rs to change their dependency specification to likely turn on
arrow-conversion
feature flag.