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

Fix newly found linter issues #3099

Merged
merged 1 commit into from
Jun 2, 2023
Merged

Fix newly found linter issues #3099

merged 1 commit into from
Jun 2, 2023

Conversation

S7evinK
Copy link
Contributor

@S7evinK S7evinK commented Jun 2, 2023

@S7evinK S7evinK added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Jun 2, 2023
@S7evinK S7evinK requested a review from a team as a code owner June 2, 2023 12:49
@codecov
Copy link

codecov bot commented Jun 2, 2023

Codecov Report

Patch coverage: 20.00% and no project coverage change.

Comparison is base (ea6b368) 66.18% compared to head (f6496f2) 66.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3099   +/-   ##
=======================================
  Coverage   66.18%   66.19%           
=======================================
  Files         500      500           
  Lines       53915    53916    +1     
=======================================
+ Hits        35686    35691    +5     
+ Misses      14633    14631    -2     
+ Partials     3596     3594    -2     
Flag Coverage Δ
unittests 50.02% <20.00%> (+0.02%) ⬆️

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

Impacted Files Coverage Δ
roomserver/internal/perform/perform_peek.go 44.95% <0.00%> (ø)
syncapi/streams/stream_pdu.go 82.13% <0.00%> (ø)
syncapi/routing/messages.go 66.84% <25.00%> (+0.08%) ⬆️
userapi/storage/shared/storage.go 77.34% <50.00%> (ø)

... and 7 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.

@S7evinK S7evinK merged commit d11da6e into main Jun 2, 2023
@S7evinK S7evinK deleted the fix-linter-issues branch June 2, 2023 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants