Skip to content

relevant search issue - #143

Closed
bugloper wants to merge 2 commits into
mongoid:masterfrom
bugloper:fix/relevant-search
Closed

bugloper wants to merge 2 commits into
mongoid:masterfrom
bugloper:fix/relevant-search

Conversation

@bugloper

Copy link
Copy Markdown

No description provided.

@dblock

dblock commented Aug 16, 2023

Copy link
Copy Markdown
Contributor

Been a while any work happened here. We need to get CI going in this project before we merge anything new, travis-CI is MIA. If you want to do the work (maybe copy all the GHA stuff from https://github.com/mongoid/mongoid-locker), I'd be happy to help merge.

@dblock

dblock commented Sep 28, 2023

Copy link
Copy Markdown
Contributor

@fatfitdev Mind splitting the GHA work with the sort fix? That will need a test, we can do that on top and make sure everything passes. LMK if you need any help with CI to pass (looking at https://github.com/fatfitdev/mongoid_search/tree/fix/relevant-search it's red).

@dblock dblock mentioned this pull request Feb 13, 2024
@dblock

dblock commented Feb 15, 2024

Copy link
Copy Markdown
Contributor

Thanks @yads for fixing CI/CD in #146.

This needs a rebase and some tests.

@dblock

dblock commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

Thanks @bugloper for reporting and working on the relevance-search problem, and for the early CI work. This is now superseded by merged #156, which restores the real MongoDB relevance regressions, fixes map-reduce reduction to return numeric scores, sorts results highest-relevance first, and documents the ordering change in UPGRADING.md. The sorting removal proposed here does not establish that ordering. CI modernization was handled separately in #146 and subsequent updates, so the old workflow changes are no longer needed. Closing as superseded rather than asking you to rebase this old branch.

I am an AI posting on behalf of @dblock.

@dblock dblock closed this Oct 18, 2026
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