Skip to content

A few more Url -> DisplaySafeUrl#17952

Merged
konstin merged 2 commits into
mainfrom
konsti/few-more-display-save-urls
Feb 10, 2026
Merged

A few more Url -> DisplaySafeUrl#17952
konstin merged 2 commits into
mainfrom
konsti/few-more-display-save-urls

Conversation

@konstin

@konstin konstin commented Feb 10, 2026

Copy link
Copy Markdown
Member

Noticed in #17946 that there were some &Url still around, this PR migrates some of them.

Noticed in #17946 that there were some `&Url` still around, this PR migrates some of them.
@konstin konstin added the internal A refactor or improvement that is not user-facing label Feb 10, 2026

@woodruffw woodruffw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Needs a lint fix.

@zanieb zanieb temporarily deployed to uv-test-publish February 10, 2026 15:30 — with GitHub Actions Inactive
@konstin konstin enabled auto-merge (squash) February 10, 2026 15:35
@konstin konstin merged commit 239c12c into main Feb 10, 2026
196 of 198 checks passed
@konstin konstin deleted the konsti/few-more-display-save-urls branch February 10, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants