-
Notifications
You must be signed in to change notification settings - Fork 177
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
Absorption #726
Absorption #726
Conversation
Hello @yang-ruoxi I think now you can run the workflows to test this PR. Please let me know for any problems or concerns. Thanks for your contribution. Cheers! :) |
Would be cool to get this merged! @itsshawnzhang If @yang-ruoxi enabled edits by maintainers, you should be able to push a small change to this PR yourself and trigger CI that way. |
Thanks @janosh and @yang-ruoxi. I will push some changes to this PR and run the new CI at this point. |
@itsshawnzhang Is allow edits from maintainers disabled? If so, it might be easier to add a the |
Will try this! Thanks! |
@itsshawnzhang This might be helpful.
|
This is helpful. Please allow me to look into it and I will create the check workflow for this PR. Thanks so much for the information @janosh :) |
Thanks for looking into it! Is there anything I should do from my end to merge it? |
@yang-ruoxi If you could push any change to this PR (fix a typo or improve a doc string or something like that) to retrigger CI that would allow @itsshawnzhang to make sure it passes all tests before merging. |
@yang-ruoxi The failing tests are due to pymongo v3/v4 incompatibilities. Can you merge or rebase the |
Hello @yang-ruoxi . I merged this branch since all the tests are passed in the new environmental settings (see #744). Thank you so much for your contributions. Please let us know for any other problems or concerns. Cheers! : ) |
Summary