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 fibers from rocketchat-version package #28776

Merged
merged 6 commits into from Apr 5, 2023

Conversation

KevLehman
Copy link
Contributor

@KevLehman KevLehman commented Apr 3, 2023

ARCH-944 ARCH-987

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Apr 3, 2023

Codecov Report

Merging #28776 (0275ee6) into develop (53bce23) will increase coverage by 6.79%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28776      +/-   ##
===========================================
+ Coverage    38.57%   45.36%   +6.79%     
===========================================
  Files          636      726      +90     
  Lines        12597    14278    +1681     
  Branches      2038     2211     +173     
===========================================
+ Hits          4859     6477    +1618     
- Misses        7478     7493      +15     
- Partials       260      308      +48     
Flag Coverage Δ
e2e 45.32% <ø> (+6.74%) ⬆️

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

@KevLehman KevLehman added stat: ready to merge PR tested and approved waiting for merge stat: QA skipped labels Apr 4, 2023
@kodiakhq kodiakhq bot merged commit a8aec65 into develop Apr 5, 2023
35 checks passed
@kodiakhq kodiakhq bot deleted the refactor/rocketchat-version-patch branch April 5, 2023 18:56
gabriellsh added a commit that referenced this pull request Apr 6, 2023
…fullName

* 'develop' of github.com:RocketChat/Rocket.Chat: (134 commits)
  refactor(Meteor.user): Remove all occurrences (#28647)
  refactor: Remove unused meteor packages (#28821)
  fix: check if custom field is required to apply rule (#28428)
  fix: Confirm password flow fixed and refactored (#28453)
  refactor(file.server): Remove wrapAsync and convert to TS (#28823)
  refactor(client): `openRoom` dilluted into components (#28794)
  refactor: Isolate last Promise.await calls on accounts’ code (#28817)
  refactor: `Service Configuration` async on server (#28813)
  refactor: `utils` to ts (#28818)
  refactor: Remove fibers from `rocketchat-version` package (#28776)
  refactor: convert apps api to ts (#28815)
  refactor: Remove Promise.await from importer-pendind-file.js (#28812)
  refactor: Remove promise.await from csv importer (#28810)
  refactor: Isolate Promise.await of updateOrCreateUserFromExternalService (#28811)
  regression: remove callbacks from `Meteor.callAsync` (#28809)
  refactor: Move from `meteor/http` to `meteor/fetch` - 2x (#28799)
  refactor: Remove unnecessary Tokens collection and Fibers usage (#28803)
  regression: add missing await for RocketChatFile (#28807)
  refactor: Remove fibers from Users Model (#28797)
  ci: add branch config back to checkout action (#28802)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: QA skipped stat: ready to merge PR tested and approved waiting for merge
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants