-
Notifications
You must be signed in to change notification settings - Fork 87
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
Feat support content encoding #570
Conversation
9c2fc3d
to
578ea94
Compare
8f2e04e
to
24cbcb4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #570 +/- ##
==========================================
+ Coverage 86.72% 87.27% +0.55%
==========================================
Files 11 12 +1
Lines 2041 2193 +152
==========================================
+ Hits 1770 1914 +144
- Misses 165 170 +5
- Partials 106 109 +3 ☔ View full report in Codecov by Sentry. |
bors merge |
570: Feat support content encoding r=Ja7ad a=Ja7ad # Pull Request ## Related issue Fixes #566 ## What does this PR do? This PR support content encoding `gzip`, `deflate`, `brotli`. ```go client := meilisearch.New("127.0.0.1:7700", meilisearch.WithContentEncoding(meilisearch.GzipEncoding, meilisearch.DefaultCompression)) ``` Note: This PR is not breaking changes. ## PR checklist Please check if your PR fulfills the following requirements: - [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)? - [x] Have you read the contributing guidelines? - [x] Have you made sure that the title is accurate and descriptive of the changes? Thank you so much for contributing to Meilisearch! Co-authored-by: Javad <ja7ad@live.com> Co-authored-by: Javad Rajabzadeh <ja7ad@live.com>
This PR was included in a batch that successfully built, but then failed to merge into main. It will not be retried. Additional information: {"message":"At least 1 approving review is required by reviewers with write access.","documentation_url":"https://docs.github.com/articles/about-protected-branches","status":"422"} |
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 @Ja7ad
bors merge
584: Update version for the next release (v0.29.0) r=brunoocasali a=meili-bot _This PR is auto-generated._ The automated script updates the version of meilisearch-go to a new version: "v0.29.0" --- Changelogs 👇 --- ##⚠️ Breaking changes (experimental feature) * `embedder` is now mandatory everywhere - Ensure compatibility with [Meilisearch v1.11](https://github.com/meilisearch/meilisearch/releases/tag/v1.11.0) by `@/Kerollmops` ## 🚀 Enhancements * Add Embedder.URL (#568) `@/polyfloyd` * Add multi-search federation (#563) `@/polyfloyd` * Feat support content encoding (#570) `@/Ja7ad` * Support experimental manager (#572) `@/A7bari` * Feat Language settings & search parameter (#580) `@J/a7ad` * Feat support retry pattern on status code 502, 503, 504 (#581) `@/Ja7ad` Thanks again to `@/A7bari,` `@/Ja7ad,` `@/Kerollmops` and `@/polyfloyd!` 🎉 Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>
Pull Request
Related issue
Fixes #566
What does this PR do?
This PR support content encoding
gzip
,deflate
,brotli
.Note: This PR is not breaking changes.
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!