Gum-Joe / 2Keys

A easy to setup second keyboard, designed for everyone.
GNU General Public License v3.0
11 stars 4 forks source link

CU-5wt3zd Major refactors to @twokeys/addons #84

Closed Gum-Joe closed 4 years ago

Gum-Joe commented 4 years ago

See ClickUp task for a full list of changes. This PR contains refactors to @twokeys/addons (mainly) as well more general refactors, in preparation for milestone 0 and the start of developing the OOBE code.

NB: default commit is a squash and merge, DO NOT DO THIS - use a regular merge commit instead

Gum-Joe commented 4 years ago

Task linked: CU-5wt3zd Registry and software refactorings

codecov[bot] commented 4 years ago

Codecov Report

Merging #84 into v1 will increase coverage by 3.46%. The diff coverage is 97.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##               v1      #84      +/-   ##
==========================================
+ Coverage   91.32%   94.79%   +3.46%     
==========================================
  Files          38       39       +1     
  Lines        1407     1344      -63     
  Branches      222      209      -13     
==========================================
- Hits         1285     1274      -11     
+ Misses        122       70      -52     
Impacted Files Coverage Δ
...ackages/@twokeys/addons/src/util/zip-downloader.ts 100.00% <ø> (+20.83%) :arrow_up:
...twokeys/server/src/oobe/software/zip-downloader.ts 100.00% <ø> (+16.12%) :arrow_up:
packages/@twokeys/addons/src/registry.ts 90.55% <96.29%> (+3.92%) :arrow_up:
packages/@twokeys/addons/src/dev-tools.ts 100.00% <100.00%> (+57.14%) :arrow_up:
...s/@twokeys/addons/src/module-interfaces/twokeys.ts 100.00% <100.00%> (ø)
...ges/@twokeys/addons/src/software-query-provider.ts 96.70% <100.00%> (-0.24%) :arrow_down:
packages/@twokeys/addons/src/software.ts 99.01% <100.00%> (ø)
packages/@twokeys/addons/src/util/constants.ts 100.00% <100.00%> (ø)
packages/@twokeys/addons/src/util/copy-contents.ts 91.86% <100.00%> (+1.27%) :arrow_up:
packages/@twokeys/addons/src/util/downloader.ts 94.73% <100.00%> (+17.65%) :arrow_up:
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 976d96f...d5de384. Read the comment docs.

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert and fixes 1 when merging d017eb8f8d7b8c1849e65573cd4c5534ff33aadb into 976d96f59adea6bba7e35b04bf0f0e5d32b26a35 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert and fixes 1 when merging 7f76f0b30d6753340c693bde79897c99e23c6868 into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging 57c0da0ff10058a9d7237fefbeafdbbc78480ce4 into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging d3fe05a214e81c503b3e49c4376c18273006953d into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging 219c76ebb5365bdc3ee34ed252da74555d189081 into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging f2de43493defb03096ab6d49c3e8de4e13bfbd08 into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging d5de3842ec31e5d74ca368cf5f475f98dc8aed9f into 2781afd0db87fc85eb9940aef70791239353521b - view on LGTM.com

fixed alerts:

Gum-Joe commented 4 years ago

Ok then I guess, you can merge it