Skip to content

PHPLIB-945: Update ChangeStreamEvent type definition to include clusterTime #971

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

Merged
merged 1 commit into from
Sep 27, 2022

Conversation

levon80999
Copy link
Contributor

@levon80999 levon80999 commented Sep 13, 2022

@jmikola
Copy link
Member

jmikola commented Sep 13, 2022

Please add the sync commit (mongodb/specifications@d9a0e7b) to your commit message.

@jmikola
Copy link
Member

jmikola commented Sep 13, 2022

Note: recent discussion in DRIVERS-2025 indicates that there will be a follow-up ticket to update this spec test. I suggest holding off on merging so that we can re-use PHPLIB-945 and this PR for the second ticket that will soon be created.

@jmikola
Copy link
Member

jmikola commented Sep 14, 2022

Per my comment in PHPLIB-945, please sync with mongodb/specifications@1a3b359.

@levon80999 levon80999 requested a review from jmikola September 14, 2022 22:57
@jmikola
Copy link
Member

jmikola commented Sep 21, 2022

@levon80999: Note that my earlier feedback from #971 (comment) has not been addressed.

@levon80999
Copy link
Contributor Author

levon80999 commented Sep 22, 2022

@jmikola sorry for late answer, I saw your comment but too busy to fix one CI fail on my other PR, I will end it today and immediately switch to this task

@levon80999 levon80999 closed this Sep 22, 2022
@levon80999 levon80999 reopened this Sep 22, 2022
@levon80999 levon80999 force-pushed the PHPLIB-945 branch 3 times, most recently from 0f93033 to 44f7095 Compare September 23, 2022 15:07
@alcaeus alcaeus removed the request for review from jmikola September 27, 2022 09:52
@levon80999 levon80999 merged commit d71588e into mongodb:master Sep 27, 2022
@levon80999 levon80999 deleted the PHPLIB-945 branch September 27, 2022 10:45
levon80999 added a commit to levon80999/mongo-php-library that referenced this pull request Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants