-
Notifications
You must be signed in to change notification settings - Fork 108
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
Changed SearchRequest
initializations to builder in code-sample
#567
Conversation
…uctor - Changed code-samples.meilisearch.yaml file, SearchRequestTest file and SearchTest file
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.
Thanks @PotatoDoge for this PR ❤️
I left a few comments. Also, we could change the SearchTest.java
file but not the src/test/java/com/meilisearch/sdk/SearchRequestTest.java because we need to check if the initialization of SearchRequest
work with both way with the builder and with the constructor.
Don't forget to run bash ./scripts/lint.sh
for fixing the linter issues 😊
…ructor - Changed code-samples.meilisearch.yaml file, and SearchTest file - Fixed linting issues
@alallema thank you for the feedback! I believe that I fixed the issues. I'll wait for you to check it. :) |
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.
Perfect ✨ Thank you @PotatoDoge
LGTM!
SearchRequest
initializations to builder in code-sample
bors merge |
Build succeeded:
|
Changed SearchRequest initializations to builder insted of a constructor
Pull Request
Related issue
Fixes #<560
What does this PR do?
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!