Skip to content

Commit

Permalink
bug(python): add bandit_configuration parameter to python configurati…
Browse files Browse the repository at this point in the history
…on initialization (FF-3390)
  • Loading branch information
leoromanovsky committed Oct 15, 2024
1 parent 427fed8 commit c46f027
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
18 changes: 17 additions & 1 deletion python-sdk/python/eppo_client/_eppo_client.pyi
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,23 @@ def init(config: ClientConfig) -> EppoClient: ...
def get_instance() -> EppoClient: ...

class Configuration:
def __init__(self, flags_configuration: bytes) -> None: ...
# Note: Before publshing v5.0.0
#
# Users current have to initialize the configuration like this:
#
# Configuration(some_bytes, some_other_bytes)
#
# In v5.0.0, we will add a named parameter to the constructor to make this
# more clear.
#
# Users will be able to initialize the configuration like this:
#
# Configuration(flags_configuration=some_bytes, bandits_configuration=some_other_bytes)
#
# Changes:
#
# def __init__(self, *, flags_configuration: bytes, bandits_configuration: bytes | None = None) -> None: ...
def __init__(self, flags_configuration: bytes, bandits_configuration: bytes | None = None) -> None: ...
def get_flags_configuration(self) -> bytes: ...
def get_flag_keys(self) -> Set[str]: ...
def get_bandit_keys(self) -> Set[str]: ...
Expand Down

0 comments on commit c46f027

Please sign in to comment.