Skip to content

Commit f9d7c4d

Browse files
committed
feat: enable git extension in desktop app by default
1 parent 4115492 commit f9d7c4d

File tree

2 files changed

+19
-2
lines changed

2 files changed

+19
-2
lines changed

src/extensions/default/DefaultExtensions.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@
2525
"UrlCodeHints",
2626
"HealthData"
2727
],
28+
"desktopOnly": [
29+
"Git"
30+
],
2831
"warnExtensionStoreExtensions": {
2932
"description": "list extension ids here that you want to show this warning in extension store: 'You may not need this extension. Phoenix comes built in with this feature.'",
3033
"note": "This is only to for legacy extensions that havent been updated for a long time, remove flag unconditionally if instructed by the extension author, but inform of the issues in extension.",

src/utils/ExtensionLoader.js

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,22 @@ define(function (require, exports, module) {
5454
UrlParams = require("utils/UrlParams").UrlParams,
5555
NodeUtils = require("utils/NodeUtils"),
5656
PathUtils = require("thirdparty/path-utils/path-utils"),
57-
DefaultExtensionsList = JSON.parse(require("text!extensions/default/DefaultExtensions.json"))
58-
.defaultExtensionsList;
57+
DefaultExtensions = JSON.parse(require("text!extensions/default/DefaultExtensions.json"));
58+
59+
const desktopOnlyExtensions = DefaultExtensions.desktopOnly;
60+
const DefaultExtensionsList = Phoenix.isNativeApp ?
61+
[...DefaultExtensions.defaultExtensionsList, ...desktopOnlyExtensions]:
62+
DefaultExtensions.defaultExtensionsList;
63+
64+
if(Phoenix.isTestWindow) {
65+
// we dont load the heavy weight git extension by default for tests as huge number
66+
// of tests written before git integration and too hard to fix those failing tests for now.
67+
// we will just have new tests from git specific workflows.
68+
const index = DefaultExtensionsList.indexOf("Git");
69+
if(index !== -1) {
70+
DefaultExtensionsList.splice(index, 1);
71+
}
72+
}
5973

6074
const customExtensionLoadPaths = {};
6175

0 commit comments

Comments
 (0)