-
Notifications
You must be signed in to change notification settings - Fork 524
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
Add optional depth parameter to SDK file listing methods. #649
base: main
Are you sure you want to change the base?
Conversation
* update js-sdk file.list method to include optional depth param * update js-sdk tests
|
Also, add docstrings for the new I'll document the spec update process so it is not blocking. |
This is blocked by the unpredictable Python client generation, right? |
@ValentaTomas yes, per discussion, im on a side-quest to build a predictable codegen, will apply it here and commit. |
…e-files-list-in-the-sdk-e2b-1902_2
Description
Do not merge before e2b-dev/infra#524 is merged and deployed.
The DX would look like this:
envd
pb specjs-sdk
js-sdk
file.list
method to include optional depth paramjs-sdk
testspython-sdk
python-sdk
file.list
method to include optional depth parampython-sdk
tests