Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Allow client event serialization to be async #5183

Merged
merged 3 commits into from
May 15, 2019

Conversation

erikjohnston
Copy link
Member

This is in preparation for bundling in reactions

@erikjohnston erikjohnston force-pushed the erikj/async_serialize_event branch from 6229801 to b54b03f Compare May 14, 2019 10:58
@erikjohnston erikjohnston force-pushed the erikj/async_serialize_event branch from d118342 to a80e6b5 Compare May 14, 2019 12:12
@codecov

This comment has been minimized.

@codecov
Copy link

codecov bot commented May 14, 2019

Codecov Report

Merging #5183 into develop will increase coverage by 0.01%.
The diff coverage is 76.38%.

@@             Coverage Diff             @@
##           develop    #5183      +/-   ##
===========================================
+ Coverage    62.15%   62.16%   +0.01%     
===========================================
  Files          336      336              
  Lines        34659    34686      +27     
  Branches      5694     5679      -15     
===========================================
+ Hits         21541    21562      +21     
- Misses       11583    11590       +7     
+ Partials      1535     1534       -1

@erikjohnston erikjohnston requested a review from a team May 14, 2019 12:18
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm modulo docstring nitpick

synapse/util/async_helpers.py Outdated Show resolved Hide resolved
@erikjohnston erikjohnston merged commit 0aba6c8 into develop May 15, 2019
@erikjohnston erikjohnston deleted the erikj/async_serialize_event branch January 9, 2020 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants