Skip to content

Feature/gzip encode request#32

Merged
whatwewant merged 2 commits intogo-zoox:masterfrom
cedrickah:feature/gzip-encode-request
Jan 18, 2026
Merged

Feature/gzip encode request#32
whatwewant merged 2 commits intogo-zoox:masterfrom
cedrickah:feature/gzip-encode-request

Conversation

@cedrickah
Copy link
Contributor

I noticed that there is a need to support request compression via gzip. Here is my attempt, including testing.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 15827339648

Details

  • 19 of 23 (82.61%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 71.718%

Changes Missing Coverage Covered Lines Changed/Added Lines %
execute.go 10 14 71.43%
Totals Coverage Status
Change from base Build 15814746870: 0.1%
Covered Lines: 743
Relevant Lines: 1036

💛 - Coveralls

@cedrickah
Copy link
Contributor Author

@whatwewant , sorry for the mention. Can you check this please?

@whatwewant
Copy link
Contributor

@cedrickah LGTM, thanks too much.

@whatwewant whatwewant merged commit 1ef7e77 into go-zoox:master Jan 18, 2026
1 check passed
@cedrickah
Copy link
Contributor Author

@cedrickah LGTM, thanks too much.

You're welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants