Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes related to the next MeiliSearch release (v0.21.0) #169

Merged
merged 20 commits into from
Aug 24, 2021

Conversation

meili-bot
Copy link
Contributor

@meili-bot meili-bot commented Apr 27, 2021

Related to this issue: meilisearch/integration-guides#117

This PR:

  • gathers the changes related to the next MeiliSearch release (v0.21.0) so that this package is ready when the official release is out.
  • should pass the tests against the latest pre-release of MeiliSearch. Until there is an RC version, the tests should be valid against the alpha versions of transplant.
  • might eventually contain test failures until the MeiliSearch v0.21.0 is out.

⚠️ This PR should NOT be merged until the next release of MeiliSearch (v0.21.0) is out.

This PR is auto-generated for the pre-release week purpose. Exceptionally for this release, the pre-release time will be more than just one week.


Done:

@meili-bot meili-bot added the skip-changelog The PR will not appear in the release changelogs label Apr 27, 2021
* Fix test by renaming buildDate into commitDate

* Improve tests according to alpha 4
@curquiza curquiza removed the skip-changelog The PR will not appear in the release changelogs label Apr 27, 2021
@curquiza curquiza added the breaking-change The related changes are breaking for the users label Jun 29, 2021
@curquiza curquiza requested a review from bidoubiwa August 23, 2021 15:28
@curquiza curquiza marked this pull request as ready for review August 23, 2021 16:32
@curquiza
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Aug 23, 2021
@bors
Copy link
Contributor

bors bot commented Aug 23, 2021

bidoubiwa
bidoubiwa previously approved these changes Aug 24, 2021
Copy link
Contributor

@bidoubiwa bidoubiwa left a comment

Choose a reason for hiding this comment

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

🔥

CONTRIBUTING.md Outdated
@@ -8,7 +8,6 @@ First of all, thank you for contributing to MeiliSearch! The goal of this docume
- [How to Contribute](#how-to-contribute)
- [Development Workflow](#development-workflow)
- [Git Guidelines](#git-guidelines)
- [Release Process (for internal team only)](#release-process-for-internal-team-only)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this wanted? Just to be sure. I don't know if it is written somewhere in which case my bad

@bidoubiwa
Copy link
Contributor

Not sure if this test should be updated by removing wordsPosition

// q_spec.rb
  it 'does a basic search with an empty query and a custom ranking rule' do
    response = @index.update_ranking_rules([
                                             'typo',
                                             'words',
                                             'proximity',
                                             'attribute',
                                             'wordsPosition',
                                             'exactness',
                                             'asc(objectId)'
                                           ])
    @index.wait_for_pending_update(response['updateId'])
    response = @index.search('')
    expect(response['nbHits']).to eq(@documents.count)
    expect(response['hits'].first['objectId']).to eq(1)
  end

Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

I applied your changes, thanks @bidoubiwa!

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 24, 2021

@bors bors bot merged commit 3d0cefa into main Aug 24, 2021
@bors bors bot deleted the bump-meilisearch-v0.21.0 branch August 24, 2021 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants