-
Notifications
You must be signed in to change notification settings - Fork 224
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
[MERGE-WITH-SDK-ROLL-RC6] [FEAT] Rolling out RC6 to OSS #889
Conversation
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.
Looks good. Hope the pdm.lock files are verified once to make sure SDK is on rc6.
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.
The version bumps LGTM however please take care of the comment added around raising the exception
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.
LGTM - left a minor comment
Signed-off-by: harini-venkataraman <[email protected]>
|
Quality Gate passedIssues Measures |
What
PR to roll out RC6 to OSS
...
Why
To enable k8s, migrating to remote storage
...
How
This changes uses the FS APIs.
...
Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
The changes are under feature flag
...
Database Migrations
Not applicable
...
Env Config
Not applicable
...
Notes on Testing
Happy path with RC6 installed in venv is tested
Checklist
I have read and understood the Contribution Guidelines.