Skip to content

Conversation

vkarpov15
Copy link
Collaborator

Summary

#15640 pointed out that the noListener option currently throws an error, so we should remove support for that in 9.0

Examples

@vkarpov15 vkarpov15 added this to the 9.0 milestone Sep 22, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the deprecated noListener option from connection's useDb() method as part of a breaking change for version 9.0, since the option currently throws an error and is no longer functional.

  • Removes noListener parameter documentation from JSDoc comments
  • Eliminates conditional logic that handled the noListener option
  • Updates TypeScript test to remove usage of the deprecated option

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
lib/connection.js Removes JSDoc documentation for the deprecated noListener parameter
lib/drivers/node-mongodb-native/connection.js Removes noListener parameter documentation and related conditional logic from useDb() implementation
test/types/connection.test.ts Removes TypeScript test case that used the deprecated noListener option

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@hasezoey hasezoey left a comment

Choose a reason for hiding this comment

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

This change should likely be mentioned in the changelog.

@vkarpov15 vkarpov15 merged commit ac9c4b5 into Automattic:9.0 Sep 28, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants