Skip to content

Conversation

@dougfabris
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@dougfabris dougfabris marked this pull request as ready for review August 12, 2022 21:14
@dougfabris dougfabris requested review from a team as code owners August 12, 2022 21:14
@dougfabris dougfabris changed the title Chore: Rewrite RoomMembers to Typescript Chore: Refactor RoomMembers to Typescript Aug 12, 2022
@ggazzo ggazzo removed the request for review from a team August 17, 2022 17:06
@dougfabris dougfabris requested a review from gabriellsh August 18, 2022 19:52
@RocketChat RocketChat deleted a comment from lgtm-com bot Aug 18, 2022
@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #26559 (634a59a) into develop (f71c273) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           develop   #26559    +/-   ##
=========================================
  Coverage    38.67%   38.68%            
=========================================
  Files          758      751     -7     
  Lines        18783    18670   -113     
  Branches      1453     1453            
=========================================
- Hits          7265     7223    -42     
+ Misses       11302    11227    -75     
- Partials       216      220     +4     
Flag Coverage Δ
e2e 38.68% <ø> (+<0.01%) ⬆️

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

@ggazzo ggazzo added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Aug 19, 2022
@kodiakhq kodiakhq bot merged commit 6e3306b into develop Aug 20, 2022
@kodiakhq kodiakhq bot deleted the chore/room-members branch August 20, 2022 20:22
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants