Skip to content

ui/profile/src/SimpleMatchers: Sort matches with levenshtein#5568

Merged
metalmatze merged 1 commit into
mainfrom
ui-simplematchers-levenshtein
Mar 12, 2025
Merged

ui/profile/src/SimpleMatchers: Sort matches with levenshtein#5568
metalmatze merged 1 commit into
mainfrom
ui-simplematchers-levenshtein

Conversation

@metalmatze
Copy link
Copy Markdown
Member

@metalmatze metalmatze commented Mar 7, 2025

Previously app_kubernetes_io_component would have been at the top. But this isn't closest to the search term anymore so we show it last.

Screenshot 2025-03-07 at 12 14 29

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Mar 7, 2025

🤖 Meticulous spotted visual differences in 7 of 360 screens tested: view and approve differences detected.

Meticulous simulated ~4 hours of user flows against your PR.

Last updated for commit 21dd619. This comment will update as new commits are pushed.

Copy link
Copy Markdown
Contributor

@yomete yomete left a comment

Choose a reason for hiding this comment

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

lgtm

@metalmatze metalmatze enabled auto-merge March 12, 2025 09:27
@metalmatze metalmatze merged commit a0cc202 into main Mar 12, 2025
@metalmatze metalmatze deleted the ui-simplematchers-levenshtein branch March 12, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants