Skip to content

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

Merged
merged 3 commits into from
Jun 29, 2021
Merged

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented May 21, 2021

PHPLIB-654

Tests against my test version, which I'll remove before merging.

@alcaeus alcaeus requested a review from jmikola May 21, 2021 13:47
@alcaeus alcaeus self-assigned this May 21, 2021
Copy link
Member

@jmikola jmikola left a 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.

@alcaeus
Copy link
Member Author

alcaeus commented May 26, 2021

Updated to reflect review feedback. Waiting for mongodb/mongo-php-driver#1221 to be merged before merging.

@alcaeus alcaeus changed the title PHPLIB-654 Remove deprecated terminology [WAIT] PHPLIB-654 Remove deprecated terminology May 26, 2021
@alcaeus alcaeus changed the title [WAIT] PHPLIB-654 Remove deprecated terminology PHPLIB-654 Remove deprecated terminology Jun 25, 2021
Copy link
Member

@jmikola jmikola left a 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.

alcaeus added 2 commits June 29, 2021 08:25
Updates spec tests to commit mongodb/specifications#6d7fd374ca14a80edabeefdd27cdd6187c43a0eb
@alcaeus
Copy link
Member Author

alcaeus commented Jun 29, 2021

Build failures are addressed in #835, merging despite failing CI.

@alcaeus alcaeus merged commit a295cd0 into mongodb:master Jun 29, 2021
@alcaeus alcaeus deleted the phplib-654 branch June 29, 2021 08:30
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.

2 participants