Changed regex #125

Merged
marcdeb1 merged 1 commit from search_accuracy into master 2018-11-07 19:50:04 +01:00
marcdeb1 commented 2018-10-24 12:24:49 +02:00 (Migrated from github.com)

See issue #119.

Could be better to add a custom analyzer with minimum word length and stop words.

See issue #119. Could be better to add a custom analyzer with minimum word length and stop words.
tzarebczan commented 2018-10-31 18:18:25 +01:00 (Migrated from github.com)

Thanks for the PR @marcdeb1 ! We'll get this reviewed. Per the usual procedure, send us an email once this is merged :)

Thanks for the PR @marcdeb1 ! We'll get this reviewed. Per the usual procedure, send us an email once this is merged :)
tiger5226 commented 2018-11-01 02:03:30 +01:00 (Migrated from github.com)

I will merge this soon. I just need to QA test it since it impacts a critical part ( the query ) and has implications around the effective amount. @tzarebczan I also build out some auto deployment so we can deploy faster.

I will merge this soon. I just need to QA test it since it impacts a critical part ( the query ) and has implications around the effective amount. @tzarebczan I also build out some auto deployment so we can deploy faster.
tiger5226 (Migrated from github.com) approved these changes 2018-11-07 19:48:32 +01:00
tiger5226 (Migrated from github.com) left a comment

looks good to me.

looks good to me.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lighthouse.js#125
No description provided.