Skip to content

refactor(NODE-7302): remove dependency on uri.parse() #4798

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 18, 2025
Merged

refactor(NODE-7302): remove dependency on uri.parse() #4798

merged 3 commits into from
Nov 18, 2025

Conversation

@durran
Copy link
Member

@durran durran commented Nov 18, 2025 •
edited
Loading

Description

Removes the dead util request code and uri import.

Summary of Changes

  • Removes the dead code

What is the motivation for this change?

NODE-7302

Release Highlight

Release notes highlight

Double check the following

  • Lint is passing (npm run check:lint)
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@durran durran changed the title feat(NODE-7302): remove dependency on uri.parse() refaactor(NODE-7302): remove dependency on uri.parse() Nov 18, 2025
@durran durran changed the title refaactor(NODE-7302): remove dependency on uri.parse() refactor(NODE-7302): remove dependency on uri.parse() Nov 18, 2025
@durran durran marked this pull request as ready for review November 18, 2025 17:09
@durran durran requested a review from a team as a code owner November 18, 2025 17:09
@baileympearson baileympearson self-assigned this Nov 18, 2025
@baileympearson baileympearson added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Nov 18, 2025
@baileympearson baileympearson merged commit 49c5b6f into main Nov 18, 2025
27 of 29 checks passed
@baileympearson baileympearson deleted the NODE-7302 branch November 18, 2025 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@baileympearson baileympearson baileympearson approved these changes

Assignees

@baileympearson baileympearson

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@durran @baileympearson