-
Notifications
You must be signed in to change notification settings - Fork 44
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
Add csv delimiter to Index#add_documents_csv
method
#429
Conversation
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.
not sure about the changes in rubocop but the rest LGTM
Unfortunately, the rubocop rules are always affected, even with minimal change, because we are infringing the main rules like class and method sizes. Without a significant refactor, this will be endless. |
432: Update version for the next release (v0.23.0) r=bidoubiwa a=meili-bot This version introduces features released on Meilisearch v1.1.0 🎉 Check out the changelog of [Meilisearch v1.1.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.1.0) for more information on the changes. ### 🚀 Enhancements - Add a new optional argument to `add_documents_csv`. This argument allows you to customize the separator character in your `csv` file. (#429) `@brunoocasali.` - Add `client.multi_search()` method to execute multiple search requests simultaneously with different configurations. (#430) `@brunoocasali` Usage example: ```ruby client.multi_search([ { index_uid: 'books', q: 'prince' }, { index_uid: 'movies', q: 'prince' }, ]) ```⚠️ The `SearchQuery` was not meant to be used if the regular `$index->search()` requests (yet). Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>
425: Changes related to the next Meilisearch release (v1.1.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#251 This PR: - gathers the changes related to the next Meilisearch release (v1.1.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.1.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.1.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ 432: Update version for the next release (v0.23.0) r=curquiza a=meili-bot This version introduces features released on Meilisearch v1.1.0 🎉 Check out the changelog of [Meilisearch v1.1.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.1.0) for more information on the changes. ### 🚀 Enhancements - Add a new optional argument to `add_documents_csv`. This argument allows you to customize the separator character in your `csv` file. (#429) `@brunoocasali.` - Add `client.multi_search()` method to execute multiple search requests simultaneously with different configurations. (#430) `@brunoocasali` Usage example: ```ruby client.multi_search([ { index_uid: 'books', q: 'prince' }, { index_uid: 'movies', q: 'prince' }, ]) ```⚠️ The `SearchQuery` was not meant to be used if the regular `$index->search()` requests (yet). Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com>
432: Update version for the next release (v0.23.0) r=curquiza a=meili-bot This version introduces features released on Meilisearch v1.1.0 🎉 Check out the changelog of [Meilisearch v1.1.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.1.0) for more information on the changes. ### 🚀 Enhancements - Add a new optional argument to `add_documents_csv`. This argument allows you to customize the separator character in your `csv` file. (#429) `@brunoocasali.` - Add `client.multi_search()` method to execute multiple search requests simultaneously with different configurations. (#430) `@brunoocasali` Usage example: ```ruby client.multi_search([ { index_uid: 'books', q: 'prince' }, { index_uid: 'movies', q: 'prince' }, ]) ```⚠️ The `SearchQuery` was not meant to be used if the regular `$index->search()` requests (yet). Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>
432: Update version for the next release (v0.23.0) r=bidoubiwa a=meili-bot This version introduces features released on Meilisearch v1.1.0 🎉 Check out the changelog of [Meilisearch v1.1.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.1.0) for more information on the changes. ### 🚀 Enhancements - Add a new optional argument to `add_documents_csv`. This argument allows you to customize the separator character in your `csv` file. (#429) `@brunoocasali.` - Add `client.multi_search()` method to execute multiple search requests simultaneously with different configurations. (#430) `@brunoocasali` Usage example: ```ruby client.multi_search([ { index_uid: 'books', q: 'prince' }, { index_uid: 'movies', q: 'prince' }, ]) ```⚠️ The `SearchQuery` was not meant to be used if the regular `$index->search()` requests (yet). Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com>
add_documents_csv
to allow a different separator.