Skip to content
This repository was archived by the owner on Nov 25, 2024. It is now read-only.

Conversation

devonh
Copy link
Collaborator

@devonh devonh commented Jun 7, 2023

No description provided.

@devonh devonh requested a review from a team as a code owner June 7, 2023 00:25
@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

Patch coverage: 25.23% and project coverage change: -0.44 ⚠️

Comparison is base (7a1fd7f) 66.20% compared to head (b2df341) 65.77%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3105      +/-   ##
==========================================
- Coverage   66.20%   65.77%   -0.44%     
==========================================
  Files         500      500              
  Lines       54118    54352     +234     
==========================================
- Hits        35829    35749      -80     
- Misses      14677    14992     +315     
+ Partials     3612     3611       -1     
Flag Coverage Δ
unittests 49.94% <25.23%> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
clientapi/routing/directory.go 46.80% <0.00%> (-4.79%) ⬇️
clientapi/routing/redaction.go 39.49% <0.00%> (-10.99%) ⬇️
clientapi/routing/sendevent.go 64.68% <0.00%> (-6.03%) ⬇️
clientapi/threepid/invites.go 7.00% <0.00%> (-0.28%) ⬇️
federationapi/routing/invite.go 43.40% <0.00%> (-2.45%) ⬇️
federationapi/routing/join.go 46.03% <0.00%> (-8.91%) ⬇️
federationapi/routing/leave.go 40.49% <0.00%> (-4.93%) ⬇️
federationapi/routing/threepid.go 0.00% <0.00%> (ø)
internal/pushrules/evaluate.go 78.76% <ø> (ø)
roomserver/api/alias.go 100.00% <ø> (ø)
... and 34 more

... and 23 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@devonh devonh merged commit 8ea1a11 into main Jun 7, 2023
@devonh devonh deleted the sender-keys branch June 7, 2023 17:14
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.

3 participants