EdgeTX / buddy

The next generation tool for EdgeTX. A cross platform app, with browser compatibility.
https://buddy.edgetx.org
MIT License
37 stars 19 forks source link

feat: allow github token to be passed for e2e tests #43

Closed freshollie closed 1 year ago

freshollie commented 1 year ago

Hopefully this fixes the CI e2e issues

codecov-commenter commented 1 year ago

Codecov Report

Merging #43 (37ec272) into main (2f85d35) will increase coverage by 0.29%. The diff coverage is 67.37%.

:mega: 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             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   52.63%   52.92%   +0.29%     
==========================================
  Files          80       82       +2     
  Lines        7813     7862      +49     
  Branches      630      628       -2     
==========================================
+ Hits         4112     4161      +49     
- Misses       3692     3693       +1     
+ Partials        9        8       -1     
Impacted Files Coverage Δ
src/main/index.ts 0.00% <0.00%> (ø)
src/renderer/gql/client.ts 0.00% <0.00%> (ø)
src/shared/backend/mocks/context.ts 0.00% <0.00%> (ø)
src/webworker/types.ts 0.00% <0.00%> (ø)
src/shared/backend/graph/flash/index.ts 52.12% <20.00%> (-0.48%) :arrow_down:
src/renderer/components/firmware/CopyUrlButton.tsx 96.87% <50.00%> (-1.54%) :arrow_down:
...rer/components/firmware/DownloadFirmwareButton.tsx 69.51% <50.00%> (-0.30%) :arrow_down:
...renderer/pages/flash/steps/DeviceSelectionStep.tsx 83.33% <50.00%> (ø)
src/shared/backend/utils/ZipHTTPRangeReader.ts 62.16% <50.00%> (+0.51%) :arrow_up:
src/shared/config.ts 77.77% <53.33%> (+20.20%) :arrow_up:
... and 10 more

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

freshollie commented 1 year ago

@bigspeedfpv mind taking a look?

freshollie commented 1 year ago

Merged anyway, it's a minor change