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

@metalmatze metalmatze commented Mar 7, 2025

Copy link
Copy Markdown
Member

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

alwaysmeticulous Bot commented Mar 7, 2025

Copy link
Copy Markdown

🤖 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.

@yomete yomete left a comment

Copy link
Copy Markdown
Contributor

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