fix: correctly set default values for arrays

now `[]` will override the default value.
redhat-plumbers-in-action · Sep 26, 2024 · b9439b6 · b9439b6
1 parent 552dc0d
commit b9439b6
Show file tree

Hide file tree

Showing 5 changed files with 53 additions and 3 deletions.
diff --git a/dist/config.js b/dist/config.js
diff --git a/dist/config.js.map b/dist/config.js.map
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/config.ts b/src/config.ts
@@ -29,12 +29,20 @@ export class Config {
   static async getConfig(octokit: CustomOctokit): Promise<Config> {
     const path = getInput('config-path', { required: true });
 
+    const overwriteMerge = (
+      _defaultArray: string[],
+      configArray: string[],
+      _options?: deepmerge.Options
+    ) => configArray;
+
     const retrievedConfig = (
       await octokit.config.get({
         ...context.repo,
         path,
         defaults: configs =>
-          deepmerge.all([this.defaults, ...configs]) as Partial<Config>,
+          deepmerge.all([this.defaults, ...configs], {
+            arrayMerge: overwriteMerge,
+          }) as Partial<Config>,
       })
     ).config;