Skip to content

PHPLIB-569: Atlas Data Lake spec tests #771

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
Sep 9, 2020
Merged

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Jul 20, 2020

https://jira.mongodb.org/browse/PHPLIB-569

Tests synced with mongodb/specifications@bdc283c

This is waiting on Evergreen migration (PHPLIB-469). In the meantime, I've been testing locally with the following phpunit.xml modification:

<env name="MONGODB_URI" value="mongodb://mhuser:[email protected]:27017"/>

@jmikola jmikola changed the title PHPLIB-569: Atlas Data Lake spec tests [WIP] PHPLIB-569: Atlas Data Lake spec tests Jul 20, 2020
@jmikola jmikola changed the title [WIP] PHPLIB-569: Atlas Data Lake spec tests PHPLIB-569: Atlas Data Lake spec tests Jul 20, 2020
@jmikola jmikola marked this pull request as draft July 20, 2020 13:02
@alcaeus alcaeus marked this pull request as ready for review September 7, 2020 07:50
@alcaeus alcaeus self-assigned this Sep 7, 2020
@alcaeus
Copy link
Member

alcaeus commented Sep 7, 2020

@jmikola I know you can't review your own PR, but if you could give the evergreen changes a quick look I'd appreciate it.

Evergreen patch: https://spruce.mongodb.com/version/5f55e63a57e85a7177ac7fd7/tasks

Copy link
Member Author

@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.

Two comments/suggestions but LGTM.

@jmikola
Copy link
Member Author

jmikola commented Sep 8, 2020

LGTM

@alcaeus alcaeus merged commit 474ec83 into mongodb:master Sep 9, 2020
@jmikola jmikola deleted the phplib-569 branch September 9, 2020 12:59
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