Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## develop #3060 +/- ##
===========================================
+ Coverage 23.00% 27.52% +4.52%
===========================================
Files 52 52
Lines 11578 11580 +2
===========================================
+ Hits 2663 3187 +524
+ Misses 8915 8393 -522
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
It's a shame about the work you put in. Maybe it would be a good idea after the new release to slowly convert one part after the other to async? Maybe not all cause the problem and we can find out which parts are critical. |
|
yes, but its not reverted yet and if we have to its not lost, we have to find the real issue which seems to be difficult, see also my comment here, if you have an idea let us know ... |
by running calendar and newsfeed tests last. Additional change: allow unit tests to run parallel This is no fix of the real issue of calendar and newsfeed tests but I moved them to the end of the tests so other tests are not failing anymore. There are coming follow up PR's for the real fixes (when I find the culprits). With these change we can stay with the async changes done by @rejas and #3060 is obsolete.
4 participants
As discussed in #3050 this PR undoes all async patches until we get the timing issues (or whatever causes the tests to fail) sorted out.