Skip to content

Comments

fix: Replace the deprecated url.parse with new URL#2735

Merged
mykola-mokhnach merged 3 commits intoappium:masterfrom
mykola-mokhnach:url-parse
Feb 19, 2026
Merged

fix: Replace the deprecated url.parse with new URL#2735
mykola-mokhnach merged 3 commits intoappium:masterfrom
mykola-mokhnach:url-parse

Conversation

@mykola-mokhnach
Copy link
Contributor

No description provided.

Comment on lines +406 to +407
remotePort: number;
remoteUrl: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't these remain optional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, they should be there. We are anyway setting these settings to default values before passing them to options

@mykola-mokhnach mykola-mokhnach merged commit 4ea4537 into appium:master Feb 19, 2026
15 checks passed
@mykola-mokhnach mykola-mokhnach deleted the url-parse branch February 19, 2026 16:33
github-actions bot pushed a commit that referenced this pull request Feb 19, 2026
## [10.23.3](v10.23.2...v10.23.3) (2026-02-19)

### Bug Fixes

* Replace the deprecated url.parse with `new URL` ([#2735](#2735)) ([4ea4537](4ea4537))
@github-actions
Copy link
Contributor

🎉 This PR is included in version 10.23.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants