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

Conversation

@kerryarchibald
Copy link
Contributor

@kerryarchibald kerryarchibald commented Mar 10, 2022


This change is marked as an internal change (Task), so will not be included in the changelog.

Preview: https://pr8026--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

@kerryarchibald kerryarchibald added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Mar 10, 2022
@kerryarchibald kerryarchibald requested a review from a team as a code owner March 10, 2022 18:22
@codecov
Copy link

codecov bot commented Mar 10, 2022

Codecov Report

Merging #8026 (ef96f5c) into develop (9082e07) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #8026   +/-   ##
========================================
  Coverage    26.66%   26.66%           
========================================
  Files          862      862           
  Lines        51954    51954           
  Branches     13159    13159           
========================================
+ Hits         13854    13856    +2     
+ Misses       38100    38098    -2     
Impacted Files Coverage Δ
src/settings/handlers/RoomSettingsHandler.ts 23.72% <0.00%> (+3.38%) ⬆️

Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

Looks awesome 🌟

@kerryarchibald kerryarchibald merged commit df6c53f into develop Mar 11, 2022
@kerryarchibald kerryarchibald deleted the kerryarchibald/ts-test-stores branch March 11, 2022 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

T-Task Refactoring, enabling or disabling functionality, other engineering tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants