Skip to content

Conversation

@cottand
Copy link
Contributor

@cottand cottand commented Jul 11, 2021

No description provided.

cottand added 2 commits July 11, 2021 13:20
…ariant

- also refactor build.gradle to not embed kotlin compiler on docs subrpoject in
order to speed up builds
- give 2GBs of heap space to gradle
@cottand cottand requested a review from williamprofit August 1, 2021 12:47
@cottand
Copy link
Contributor Author

cottand commented Aug 1, 2021

PR is bigger than initially planned but

  • write docs for serialization module
  • write docs for pollWithFlow serializing variant
  • write a serializing variant of send
    • unit test
  • write docs for serializing variant of send

@cottand cottand changed the title docs(serialization): add docs for protobuf-serliazation module docs(serialization): add docs for protobuf-serliazation module, and send serializing Aug 1, 2021
@codecov
Copy link

codecov bot commented Aug 1, 2021

Codecov Report

Merging #14 (0e73183) into master (322fd58) will increase coverage by 6.17%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   87.50%   93.67%   +6.17%     
==========================================
  Files           6        8       +2     
  Lines          40       79      +39     
  Branches        9       12       +3     
==========================================
+ Hits           35       74      +39     
  Partials        5        5              
Impacted Files Coverage Δ
...ain/kotlin/io/github/kkafka/serialization/proto.kt 100.00% <ø> (ø)
...n/kotlin/io/github/kkafka/consumer/flowProtobuf.kt 100.00% <100.00%> (ø)
...n/kotlin/io/github/kkafka/producer/sendProtobuf.kt 100.00% <100.00%> (ø)
...in/kotlin/io/github/kkafka/producer/transaction.kt 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 322fd58...0e73183. Read the comment docs.

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.

2 participants