-
Notifications
You must be signed in to change notification settings - Fork 46
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
Conversation
* Fix test by renaming buildDate into commitDate * Improve tests according to alpha 4
* Update wait_for_pending_update method * Update .rubocop_todo
* Rename fieldsDistribution * Fix rubocop error * Upadate .rubocop_todo.yml
* Fix update test * Fix settings test * Fix search tests * Fix docs tests * Fix rubocop error
* Add phrase search test * Add settings search * Update rubocop todo
bors try |
tryBuild succeeded: |
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.
🔥
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) |
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.
Is this wanted? Just to be sure. I don't know if it is written somewhere in which case my bad
Not sure if this test should be updated by removing // 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 |
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 applied your changes, thanks @bidoubiwa!
bors merge
Related to this issue: meilisearch/integration-guides#117
This PR:
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:
wait_for_pending_update
Update wait_for_pending_update method #191attributes_for_faceting
intofilterable_attributes
Rename attributes_for_faceting into filterable_attributes #192 -> BREAKING