-
Notifications
You must be signed in to change notification settings - Fork 753
Release 1.2.0 beta.1 SDK #626
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
Conversation
Co-authored-by: Vasek Mlejnsky <vasek@e2b.dev>
Co-authored-by: Vasek Mlejnsky <vasek@e2b.dev>
Co-authored-by: Vasek Mlejnsky <vasek@e2b.dev>
Co-authored-by: Vasek Mlejnsky <vasek@e2b.dev>
|
@0div can you rebase it to make here only your changes? |
|
The branch is pretty messy, so I don't think this is simple to do. |
|
@ValentaTomas, is there a specific cluster against which i can re-test this that has comprehensive beta features? |
It should be foxtrot right now (and in the future staging). |
|
@mishushakov Please review, resolve and merge when ready. |
|
@0div can you update the description? Is this basically a merge main <> beta? |
@mishushakov yes, it is essentially a merge of main (a week+ old version of it now) into beta. The description is ment to highlight what features in particular are being added to this release in particular and why. |
|
@0div did you run the tests locally? Besides that, looks good so far, also make sure OpenAPI spec is merged properly and clients are generated |
@mishushakov tested against foxtrot, made some adjustments, pls take another look |
mishushakov
left a comment
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 think this is safe to merge - please verify also that the the SDKs build locally
|
|
Okay, I think you're good to go 👍 |
Uh oh!
There was an error while loading. Please reload this page.