-
Couldn't load subscription status.
- Fork 163
Flagging SDK Dev Branch #3586
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
base: main
Are you sure you want to change the base?
Flagging SDK Dev Branch #3586
Conversation
OpenFeature recommends relying on web-sdk package instead of core. The downside is that web-sdk is not side-effect free, so it's better if we don't depend on StandardResolutionReasons. The good news is that TypeScript should be able to check literals against enum and using 'DEFAULT' literal is shorter and reduces bundle size a bit.
Remove peer dependencies from `dependencies` list. Also use caret range, so users are free to select SDK version.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3586 +/- ##
==========================================
- Coverage 92.37% 92.01% -0.36%
==========================================
Files 323 326 +3
Lines 8132 8206 +74
Branches 1840 1860 +20
==========================================
+ Hits 7512 7551 +39
- Misses 620 655 +35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Bundles Sizes Evolution
🚀 CPU Performance
🧠 Memory Performance
|
…age (#3593) * 👷[FFL-125] support npm pre-release with uuid tagging on flagging package * switch codeowners ordering so packages/flagging gets matched * pushd/popd
* FFL-1548: fix API path * fix path to package.json in integrate script * lint
* flagging sdk correctly matches configuration api * fallback to precompute api v1
* change baseUrl to site * extract out RUM * update readme and tidy params * add missing file
* custom headers and fix response shape --------- Co-authored-by: Leo Romanovsky <[email protected]>
Motivation
Changes
Test instructions
Checklist