ConfigProvider: fromEnv: add missing Partial modifier (#4554)

Author: gcanti

.changeset/shiny-pets-study.md

@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+ConfigProvider: `fromEnv`: add missing `Partial` modifier.

packages/effect/dtslint/ConfigProvider.tst.ts

@@ -0,0 +1,11 @@
+import { ConfigProvider } from "effect"
+import { describe, expect, it } from "tstyche"
+
+describe("ConfigProvider", () => {
+  describe("fromEnv", () => {
+    it("should accept a partial configuration", () => {
+      expect(ConfigProvider.fromEnv({ pathDelim: "." })).type.toBe<ConfigProvider.ConfigProvider>()
+      expect(ConfigProvider.fromEnv({ seqDelim: "." })).type.toBe<ConfigProvider.ConfigProvider>()
+    })
+  })
+})

packages/effect/src/ConfigProvider.ts

@@ -170,13 +170,17 @@ export const makeFlat: (options: {
 }) => ConfigProvider.Flat = internal.makeFlat
 
 /**
- * A config provider that loads configuration from context variables,
- * using the default System service.
+ * A config provider that loads configuration from context variables
+ *
+ * **Options**:
+ *
+ * - `pathDelim`: The delimiter for the path segments (default: `"_"`).
+ * - `seqDelim`: The delimiter for the sequence of values (default: `","`).
  *
  * @since 2.0.0
  * @category constructors
  */
-export const fromEnv: (config?: ConfigProvider.FromEnvConfig) => ConfigProvider = internal.fromEnv
+export const fromEnv: (options?: Partial<ConfigProvider.FromEnvConfig>) => ConfigProvider = internal.fromEnv
 
 /**
  * Constructs a new `ConfigProvider` from a key/value (flat) provider, where

packages/effect/src/internal/configProvider.ts

@@ -102,9 +102,9 @@ export const fromFlat = (flat: ConfigProvider.ConfigProvider.Flat): ConfigProvid
 
 /** @internal */
 export const fromEnv = (
-  config?: Partial<ConfigProvider.ConfigProvider.FromEnvConfig>
+  options?: Partial<ConfigProvider.ConfigProvider.FromEnvConfig>
 ): ConfigProvider.ConfigProvider => {
-  const { pathDelim, seqDelim } = Object.assign({}, { pathDelim: "_", seqDelim: "," }, config)
+  const { pathDelim, seqDelim } = Object.assign({}, { pathDelim: "_", seqDelim: "," }, options)
   const makePathString = (path: ReadonlyArray<string>): string => pipe(path, Arr.join(pathDelim))
   const unmakePathString = (pathString: string): ReadonlyArray<string> => pathString.split(pathDelim)