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

Introduce CLI flag for systemimage server urls #2504

Merged
merged 4 commits into from
Mar 20, 2022
Merged

Conversation

NeoTheThird
Copy link
Member

supercedes #2101.

Use npm start -- --systemimage "https://wherever.xyz" for custom settings.

@codecov
Copy link

codecov bot commented Mar 16, 2022

Codecov Report

Merging #2504 (f8cc576) into master (45789e1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2504   +/-   ##
=======================================
  Coverage   74.28%   74.28%           
=======================================
  Files          29       29           
  Lines         941      941           
=======================================
  Hits          699      699           
  Misses        242      242           
Impacted Files Coverage Δ
src/lib/cli.js 100.00% <ø> (ø)
src/core/plugins/systemimage/api.js 100.00% <100.00%> (ø)

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 45789e1...f8cc576. Read the comment docs.

@NeoTheThird NeoTheThird merged commit 862b7fa into master Mar 20, 2022
@NeoTheThird NeoTheThird deleted the sysimageurl branch March 20, 2022 21:17
llewelld pushed a commit to llewelld/ubports-installer that referenced this pull request Apr 26, 2022
* Allow to override default system-image server using envar

* Introduce CLI flag for systemimage server urls
supercedes ubports#2101

* Fix test

* fix more tests when running on ci

Co-authored-by: Marius Gripsgard <marius@ubports.com>
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