Skip to content

[Cosmos] updates for integrated cache PR - #25391

Merged
JericHunter merged 8 commits into
mainfrom
jh-integrated-cache
Jul 29, 2022
Merged

[Cosmos] updates for integrated cache PR#25391
JericHunter merged 8 commits into
mainfrom
jh-integrated-cache

Conversation

@JericHunter

@JericHunter JericHunter commented Jul 25, 2022

Copy link
Copy Markdown
Contributor

Removed anything that says preview or beta or provisional, etc. Also made some changelog/readme changes.

@ghost ghost added the Cosmos label Jul 25, 2022
Comment thread sdk/cosmos/azure-cosmos/CHANGELOG.md Outdated

@simorenoh simorenoh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment, other than that it's good thanks Jeric!

@JericHunter
JericHunter marked this pull request as ready for review July 25, 2022 20:45
Comment thread sdk/cosmos/azure-cosmos/CHANGELOG.md Outdated

@simorenoh simorenoh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the approval while we make changes to the README as well.

Comment thread sdk/cosmos/azure-cosmos/CHANGELOG.md
Comment thread sdk/cosmos/azure-cosmos/README.md Outdated

@simorenoh simorenoh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one minor thing but lgtm! thanks Jeric!

@JericHunter
JericHunter merged commit e29b1db into main Jul 29, 2022
@JericHunter
JericHunter deleted the jh-integrated-cache branch July 29, 2022 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants