Skip to content

Conversation

@rustagir
Copy link
Contributor

@rustagir rustagir commented Mar 10, 2025

Adds documentation for query logging
https://jira.mongodb.org/browse/DOCSP-43518

staging

Checklist

  • Add tests and ensure they pass

@rustagir rustagir requested a review from a team as a code owner March 10, 2025 18:57
@rustagir rustagir requested a review from norareidy March 10, 2025 18:57
@github-actions github-actions bot added the docs label Mar 10, 2025
@rustagir rustagir force-pushed the DOCSP-43518-logging branch from 85f86ff to 22f480c Compare March 10, 2025 18:58
Copy link
Contributor

@mongoKart mongoKart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rustagir rustagir requested a review from GromNaN March 11, 2025 13:08
@rustagir rustagir force-pushed the DOCSP-43518-logging branch from 5d4113d to c7f2766 Compare March 21, 2025 18:04
@GromNaN GromNaN force-pushed the DOCSP-43518-logging branch from 200a679 to d589eff Compare May 9, 2025 10:23
@GromNaN GromNaN requested a review from a team as a code owner May 9, 2025 10:23
@GromNaN GromNaN changed the base branch from 5.2 to 5.4 May 9, 2025 10:23
@GromNaN
Copy link
Member

GromNaN commented May 9, 2025

@rustagir I rebased the branch on 5.4, and fixed the test. I don't get why the expectOutputRegex did not work as expected with runInSeparateProcess.

@rustagir
Copy link
Contributor Author

@GromNaN thanks for fixing the test! I'll await your review on this PR (no rush)

@rustagir rustagir merged commit 7653289 into mongodb:5.4 Jun 10, 2025
71 checks passed
paulinevos pushed a commit to paulinevos/laravel-mongodb that referenced this pull request Jun 19, 2025
* DOCSP-43518: query logging

* formatting

* remove controller

* tests

* tests

* tests

* tests

* test

* test

* test

* test

* test

* formatting

* Fix test on output

---------

Co-authored-by: Jérôme Tamarelle <jerome.tamarelle@mongodb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants