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 calls from importer classes #28404

Merged
merged 3 commits into from Mar 14, 2023

Conversation

pierre-lehnen-rc
Copy link
Contributor

@pierre-lehnen-rc pierre-lehnen-rc commented Mar 13, 2023

Proposed changes (including videos or screenshots)

Issue(s)

ARCH-420
[ARCH-421]
[ARCH-422]
[ARCH-423]
[ARCH-424]

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Mar 13, 2023

Codecov Report

Merging #28404 (536e1ef) into develop (780e7a2) will decrease coverage by 1.20%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28404      +/-   ##
===========================================
- Coverage    45.13%   43.93%   -1.20%     
===========================================
  Files          758      748      -10     
  Lines        14681    14467     -214     
  Branches      2070     2056      -14     
===========================================
- Hits          6626     6356     -270     
- Misses        7762     7839      +77     
+ Partials       293      272      -21     
Flag Coverage Δ
e2e 43.89% <ø> (-1.20%) ⬇️

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

@pierre-lehnen-rc pierre-lehnen-rc changed the title refactor: Remove fibers from importer classes refactor: Remove Promise.await calls from importer classes Mar 13, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Mar 14, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 14, 2023
@kodiakhq kodiakhq bot merged commit 7532102 into develop Mar 14, 2023
35 of 36 checks passed
@kodiakhq kodiakhq bot deleted the refactor/importer-fibers branch March 14, 2023 12:35
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
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants