meilisearch / meilisearch-java

Java client for Meilisearch

Home Page:https://meilisearch.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Apply `SearchRequest` builder changes to code-sample

alallema opened this issue · comments

Following the PR #543, now the SearchRequest could be used with a builder:

SearchRequest.builder()
    .q("")
    .offset(10)
    .limit(200)
    .cropLength(5)
    .cropMarker("…")
    .build();

For consistency, all code samples must be modified using the builder method instead of:

new SearchRequest("")
    .setOffset(10)
    .setLimit(200)
    .setCropLength(5)
    .setCropMarker("…");

Or any other variants...

Could you assign this to me, please?

Hi @PotatoDoge

We prefer not assigning external people to our issues.

Why? People often ask to be assigned and never come back, which discourages the volunteer contributors from opening a PR to fix the issue.

We will accept and merge the first PR that fixes correctly and implements the issue following the contribution guidelines of the corresponding repository.

We are looking forward to your contributions. No need to wait for an assignment to start! 👍

what java version should I use? (in order to configure correctly my IDE)

Hi @PotatoDoge,
I would recommend working with Java 8

just to be clear, what must be done is to change all SearchRequest instances in the project (both testing and class files) to be initialized with the builder, right?

just to be clear, what must be done is to change all SearchRequest instances in the project (both testing and class files) to be initialized with the builder, right?

No only the one in the .code-samples.meilisearch.yaml file

Just created a pull request!! :D