-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #327 +/- ##
==========================================
+ Coverage 60.51% 60.93% +0.42%
==========================================
Files 225 226 +1
Lines 12054 12310 +256
==========================================
+ Hits 7294 7501 +207
- Misses 4760 4809 +49
☔ View full report in Codecov by Sentry. |
3b47424
to
904c23e
Compare
Signed-off-by: Simon Paitrault <[email protected]>
225362e
to
3d5b6b9
Compare
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.
One question I have is about how the system will behave when the channels are full for whatever reason. Would it be very messy to add a test for that?
Can you define the channels that you're talking about? We have many different kind |
25da63e
to
a6f376b
Compare
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.
Just a minor change: Removing the comments.
a6f376b
to
01cb570
Compare
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.
LGTM 👍
01cb570
to
885e74b
Compare
Signed-off-by: Simon Paitrault <[email protected]>
885e74b
to
d334235
Compare
Description
This PR is adding
GraphQL
subscriptions to the current API implementation. It introduces the basic functionality and creates the newTransientStream
struct to manage those new streams.PR Checklist: