-
Notifications
You must be signed in to change notification settings - Fork 1
refactor: JS SDK v5 #256
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
Open
rasendubi
wants to merge
27
commits into
main
Choose a base branch
from
refactor-new-configuration-store
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+3,345
−5,881
Open
refactor: JS SDK v5 #256
Changes from 10 commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
60fafe1
refactor: add ConfigurationStore abstraction
rasendubi 68c8b2f
refactor: add PersistentConfigurationStorage interface
rasendubi b51b576
refactor: add Configuration type (temp aliased to IConfiguration)
rasendubi a491d0b
chore: add top-level .prettierrc
rasendubi 9325c88
refactor: make evaluation use new ConfigurationStore
rasendubi c2a07c1
refactor: fix linter errors
rasendubi 6df8767
refactor: add Configuration type
rasendubi 4d3b534
refactor: make requestor work with new ConfigurationStore
rasendubi 2f70a51
refactor: eppo client cleanup
rasendubi d4c6f06
refactor: remove temp configuration
rasendubi 60c8617
refactor: don't fetch bandits if current model is up-to-date
rasendubi 51f4abb
refactor: implement new initialization/poller behavior
rasendubi 34ff6c3
refactor: introduce configuration feed
rasendubi a7da43d
Merge remote-tracking branch 'origin/main' into HEAD
rasendubi 6b4725c
feat: extend Configuration/Requestor to support precomputed
rasendubi 311f61f
feat: support precomputed config for flags evaluation
rasendubi e5e5b9f
feat: make EppoClient handle precomputed bandits
rasendubi 9ee4834
feat: add Subject (subject-scoped client)
rasendubi 3b79e68
feat: make getPrecomputedConfiguration return Configuration instead o…
rasendubi bef0393
refactor: remove old configuration store
rasendubi 311364d
refactor: revise public/internal APIs
rasendubi 9647e30
refactor: cleanup/prettier
rasendubi 4c9a8f8
refactor: remove IConfigurationWire
rasendubi 896ffb8
Merge remote-tracking branch 'origin/main' into HEAD
rasendubi a8866fb
refactor: re-run prettier
rasendubi 6bea66d
refactor: remove ConfigDetails
rasendubi badd643
refactor: fix lints
rasendubi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
{ | ||
"singleQuote": true, | ||
"trailingComma": "all", | ||
"printWidth": 100 | ||
} |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
🧹