Skip to content

Linux app update was broken with new builds #2927

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.

Sign up for GitHub

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

Merged
abose merged 2 commits into from
May 18, 2026

refactor(updater): run Linux installer in external terminal

e95582a
Select commit
Loading
Failed to load commit list.
Merged

Linux app update was broken with new builds #2927

refactor(updater): run Linux installer in external terminal
e95582a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 18, 2026 in 41s

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 319 in src/extensionsIntegrated/appUpdater/update-electron.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ46UpzuHyptNfSkv_E8&open=AZ46UpzuHyptNfSkv_E8&pullRequest=2927

Check warning on line 363 in src/extensionsIntegrated/appUpdater/update-electron.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ46UpzuHyptNfSkv_E9&open=AZ46UpzuHyptNfSkv_E9&pullRequest=2927
View more details on SonarQubeCloud