Conversation
|
@h1alexbel thank you for your Pull Request. I'll assign someone to review it soon. If this PR solves a |
|
@l3r8yJ please review this Pull Request. Deadline (when it should be merged or closed) is You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough. Estimation here is |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #333 +/- ##
=========================================
Coverage 96.27% 96.27%
Complexity 126 126
=========================================
Files 61 61
Lines 295 295
=========================================
Hits 284 284
Misses 11 11 ☔ View full report in Codecov by Sentry. |
|
@zoeself deregister |
@h1alexbel ok, I've removed this task from scope. I'm not managing it anymore. |
|
@rultor merge |
@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here |
@h1alexbel Done! FYI, the full log is here (took me 2min) |
closes #306
PR-Codex overview
This PR introduces Kafka Integration Tests and deprecates EntryTest. It also adds KafkaRunsTest, ConsumerTest, and ProducerTest classes.
Detailed summary