Skip to content
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

ChangeFeedProcessor: Refactors AVAD metadata Contract #4380

Conversation

philipthomas-MSFT
Copy link
Contributor

Pull Request Template

Description

Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change.

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

Closing issues

To automatically close an issue: closes #IssueNumber

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

All good!

@philipthomas-MSFT philipthomas-MSFT changed the title ChangeFeedProcessor: Updates AVAD metadata Contract ChangeFeedProcessor: Refactors AVAD metadata Contract Mar 29, 2024
@philipthomas-MSFT philipthomas-MSFT self-assigned this Mar 29, 2024
@philipthomas-MSFT philipthomas-MSFT added the auto-merge Enables automation to merge PRs label Mar 29, 2024
Copy link
Contributor

@jcocchi jcocchi left a comment

Choose a reason for hiding this comment

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

LGTM!

@microsoft-github-policy-service microsoft-github-policy-service bot merged commit 4855a6a into master Mar 29, 2024
21 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot deleted the users/philipthomas-MSFT/refactor_changeFeedItemChange_to_changeFeedItem branch March 29, 2024 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enables automation to merge PRs
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants