Skip to content

Convert translator callbacks to async/await#3048

Merged
khassel merged 4 commits intoMagicMirrorOrg:developfrom
rejas:misc/async_translator
Feb 21, 2023
Merged

Convert translator callbacks to async/await#3048
khassel merged 4 commits intoMagicMirrorOrg:developfrom
rejas:misc/async_translator

Conversation

@rejas
Copy link
Collaborator

@rejas rejas commented Feb 21, 2023

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #3048 (e55bdc5) into develop (6d86ffa) will increase coverage by 0.02%.
The diff coverage is 0.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##           develop    #3048      +/-   ##
===========================================
+ Coverage    22.90%   22.93%   +0.02%     
===========================================
  Files           52       52              
  Lines        11546    11533      -13     
===========================================
  Hits          2645     2645              
+ Misses        8901     8888      -13     
Impacted Files Coverage Δ
js/loader.js 0.00% <0.00%> (ø)
js/main.js 0.00% <0.00%> (ø)
js/module.js 0.00% <0.00%> (ø)
js/translator.js 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@khassel
Copy link
Collaborator

khassel commented Feb 21, 2023

can this be merged or will you wait for other approvals?

@rejas
Copy link
Collaborator Author

rejas commented Feb 21, 2023

can this be merged or will you wait for other approvals?

If the tests run through it can be merged right away

@khassel khassel merged commit 2b792cd into MagicMirrorOrg:develop Feb 21, 2023
@rejas rejas deleted the misc/async_translator branch February 21, 2023 22:01
rejas pushed a commit to rejas/MagicMirror that referenced this pull request Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants