allow passing single generic argument to mergeConfigs

dcastil · Aug 20, 2023 · ca450e4 · ca450e4
1 parent ea367b4
commit ca450e4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/lib/merge-configs.ts b/src/lib/merge-configs.ts
@@ -4,7 +4,7 @@ import { ConfigExtension, GenericConfig } from './types'
  * @param baseConfig Config where other config will be merged into. This object will be mutated.
  * @param configExtension Partial config to merge into the `baseConfig`.
  */
-export function mergeConfigs<ClassGroupIds extends string, ThemeGroupIds extends string>(
+export function mergeConfigs<ClassGroupIds extends string, ThemeGroupIds extends string = never>(
     baseConfig: GenericConfig,
     {
         cacheSize,

diff --git a/tests/type-generics.test.ts b/tests/type-generics.test.ts
@@ -169,4 +169,8 @@ test('mergeConfigs type generics work correctly', () => {
     const config2 = mergeConfigs<'very', 'strict'>(getDefaultConfig(), {})
 
     expect(config2).toEqual(expect.any(Object))
+
+    const config3 = mergeConfigs<'single-arg'>(getDefaultConfig(), {})
+
+    expect(config3).toEqual(expect.any(Object))
 })