Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: remove Promise.await #28539

Merged
merged 39 commits into from
Mar 22, 2023
Merged

refactor: remove Promise.await #28539

merged 39 commits into from
Mar 22, 2023

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Mar 22, 2023

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

❗ No coverage uploaded for pull request base (develop@4a6629f). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop   #28539   +/-   ##
==========================================
  Coverage           ?   45.02%           
==========================================
  Files              ?      755           
  Lines              ?    14622           
  Branches           ?     2108           
==========================================
  Hits               ?     6584           
  Misses             ?     7736           
  Partials           ?      302           
Flag Coverage Δ
e2e 44.98% <ø> (?)

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

Base automatically changed from refactor/promises-05 to develop March 22, 2023 14:46
@ggazzo ggazzo marked this pull request as ready for review March 22, 2023 15:04
@ggazzo ggazzo requested review from a team as code owners March 22, 2023 15:04
@rodrigok rodrigok merged commit e6752c8 into develop Mar 22, 2023
@rodrigok rodrigok deleted the refactor/promises-06 branch March 22, 2023 17:37
gabriellsh added a commit that referenced this pull request Mar 23, 2023
…ove/full-name-username

* 'develop' of github.com:RocketChat/Rocket.Chat: (54 commits)
  chore: ignore `.eslintcache` (#28475)
  refactor: convert omnichannel callbacks to ts (#28564)
  refactor: Allow RoomCoordinator.roomFind to be async (#28566)
  refactor(models): Use Messages Raw model (4/N) (#28558)
  refactor(client): `meteor/session` usage (1/N) (#28565)
  refactor: Don't use meteor collections on migrations (#28563)
  feat: VideoConference Guest mode and Conference Router (#28186)
  refactor: promise.await removal 8/N (#28560)
  refactor: checkUsernameAvailability to async (#28557)
  refactor(promise.await): Low hanging fruits (2/2) (#28556)
  fix: Changing the app's error verification (#28450)
  refactor(promise.await): Low hanging fruits (1/N) (#28555)
  fix: Thread message preview (#28454)
  refactor: remove Promise.await from LDAP files (#28527)
  refactor: canAccessRoomId to async (#28540)
  refactor: Remove `LivechatInquiry` model (#28487)
  refactor(models): Use Messages Raw model (3/N) (#28549)
  refactor: remove Promise.await (#28539)
  chore: Make CI fail if checks fail (#28552)
  refactor: remove Promise.await from oauth manager (#28530)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants