-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add a test where stop-words are set?
…ro in routes/settings/mod.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have an issue when using stop-words:
- add documents with
title: "the hobbit"
- add
the
as stop-words - search for "the hobbit" -> I don't get any result, I should get my document with "the hobbit" if I'm not wrong.
Might be an issue in Milli, @Kerollmops?
My bad, checked with Marin, the bump of milli made the settings change! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
bors merge
Merging since the issue mentionned by @curquiza are not relevant to transplant |
See comment above |
bors merge |
Build succeeded: |
closes #21