Skip to content

Commit

Permalink
Revert "preferences: fix PreferenceProvider.merge method (#12126)" (#…
Browse files Browse the repository at this point in the history
…12213)

This reverts commit b28c8b9.
  • Loading branch information
vince-fugnitto authored Feb 23, 2023
1 parent 8cd563e commit a3b9735
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 40 deletions.
2 changes: 1 addition & 1 deletion examples/api-tests/src/launch-preferences.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
* Expectations should be tested and aligned against VS Code.
* See https://github.com/akosyakov/vscode-launch/blob/master/src/test/extension.test.ts
*/
describe.skip('Launch Preferences', function () {
describe('Launch Preferences', function () {
this.timeout(10_000);

const { assert } = chai;
Expand Down
36 changes: 0 additions & 36 deletions packages/core/src/browser/preferences/preference-provider.spec.ts

This file was deleted.

3 changes: 0 additions & 3 deletions packages/core/src/browser/preferences/preference-provider.ts
Original file line number Diff line number Diff line change
Expand Up @@ -235,9 +235,6 @@ export abstract class PreferenceProvider implements Disposable {
if (JSONExt.isObject(source[key]) && JSONExt.isObject(value)) {
this.merge(source[key], value);
continue;
} else if (JSONExt.isArray(source[key]) && JSONExt.isArray(value)) {
source[key] = [...JSONExt.deepCopy(source[key] as any), ...JSONExt.deepCopy(value)];
continue;
}
}
source[key] = JSONExt.deepCopy(value);
Expand Down

0 comments on commit a3b9735

Please sign in to comment.