-
Notifications
You must be signed in to change notification settings - Fork 5
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
Materialize mongo alldocs collection #550
Conversation
TODO:
|
It looks like there are a couple of merge conflicts that need to be resolved on a couple of files in this PR, would you like to resolve them @PeopleMakeCulture, or should I do it? they seem fairly trivial so I'm happy to do it as well, do let me know. |
@sujaypatil96 Hey sorry for the delayed response. I'm still catching up from covid last week. Could you go ahead and resolve the conflicts? Happy to take a look if there's anything confusing |
Dependency versions updated via: #585 |
Description
Fixes #543
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Tested manually in local instance of dagster

added test_materialize_all_docs()
Configuration Details: none
Checklist:
black nmdc_runtime/
?)docs/
and in https://github.com/microbiomedata/NMDC_documentation/?)make up-test && make test-run
)