-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Computed properties Python Support #33626
Conversation
Adds the keyword for computed properties in container creation
Added keyword description for computed properties
This adds support for computed properties in the python SDK.
/azp run python - cosmos - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
API change check API changes are not detected in this pull request. |
removed code used for local testing
/azp run python - cosmos - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
Updated changelog
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.
awesome stuff, thanks Bryan! let's make sure to update the changelog and it should be good to go 👍
pipeline complained about link in changelog being broken, updated the link.
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.
let's shorten the information in the docs and we should be good, thanks!
Old docstrings descriptions were too long. Updated to better match the docstrings in java and provide a link to access further information about computed properties.
/azp run python - cosmos - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
Description
Adds support for computed properties.
Sample Usage:
Also see Feature Doc
Computed properties cannot have aggregate functions.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines