From b7297e28b273e59c24ced25f8798ea5c4a4d5138 Mon Sep 17 00:00:00 2001 From: Janne Savolainen Date: Mon, 31 Jan 2022 12:39:15 +0200 Subject: [PATCH] Remove options from writeJsonFile for YAGNI Signed-off-by: Janne Savolainen --- src/common/fs/write-json-file.injectable.ts | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/common/fs/write-json-file.injectable.ts b/src/common/fs/write-json-file.injectable.ts index 48d3d3bde3..7bef449444 100644 --- a/src/common/fs/write-json-file.injectable.ts +++ b/src/common/fs/write-json-file.injectable.ts @@ -13,19 +13,12 @@ interface Dependencies { ensureDir: (dir: string, options?: EnsureOptions | number) => Promise; } -const writeJsonFile = ({ writeJson, ensureDir }: Dependencies) => async (filePath: string, content: JsonValue, options?: WriteOptions | BufferEncoding) => { +const writeJsonFile = ({ writeJson, ensureDir }: Dependencies) => async (filePath: string, content: JsonValue) => { await ensureDir(path.dirname(filePath), { mode: 0o755 }); - const resolvedOptions = typeof options === "string" - ? { - encoding: options, - } - : options; - await writeJson(filePath, content, { encoding: "utf-8", spaces: 2, - ...resolvedOptions, }); };