-
Notifications
You must be signed in to change notification settings - Fork 266
PHPLIB-654 Remove deprecated terminology #828
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
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.
Suggestion to change example command and add a reference to the specs repo's commit hash to the test sync commit. LGTM otherwise.
Updated to reflect review feedback. Waiting for mongodb/mongo-php-driver#1221 to be merged before merging. |
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 with commit message revision.
The commit for syncing spec tests refers to alcaeus@6d7fd37, which still seems incorrect. Please update to mongodb/specifications@...
if you have the corresponding specs commit hash handy.
Updates spec tests to commit mongodb/specifications#6d7fd374ca14a80edabeefdd27cdd6187c43a0eb
Build failures are addressed in #835, merging despite failing CI. |
PHPLIB-654
Tests against my test version, which I'll remove before merging.