Skip to content

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.

🔥

- [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.

5 participants