Skip to content

Conversation

@lewis-wf
Copy link

This fixes a bug where ltd. was not being picked up when matching by type. A better solution would be making the search agnostic on ending full stops, potentially.

This fixes a bug where `ltd.` was not being picked up when matching by type. A better solution would be making the search agnostic on ending full stops, potentially.
@lewis-wf
Copy link
Author

@psolin Any chance we could get a review on this?

Copy link
Contributor

@rjurney rjurney left a comment

Choose a reason for hiding this comment

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

lgtm

@rjurney
Copy link
Contributor

rjurney commented Dec 15, 2025

@lewis-wf so sorry for the long review time but I added a TON of endings and this now has conflcits. Can you please resolve them?

@rjurney
Copy link
Contributor

rjurney commented Dec 15, 2025

@psolin can you clean this up in two seconds and merge it? We need to promptly review contributions to encourage them. I can be maintainer if you're busy - I'm chewing this dirt daily :)

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.

3 participants