🐛 ExecOptions.yank should be optional

lambdalisue · Nov 11, 2023 · c0ac358 · c0ac358
1 parent ac2d5bc
commit c0ac358
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/denops/gin/command/browse/command.ts b/denops/gin/command/browse/command.ts
@@ -15,14 +15,14 @@ import { yank } from "../../util/yank.ts";
 
 export type ExecOptions = Omit<Options, "cwd" | "aliases"> & {
   worktree?: string;
-  yank: string | boolean;
+  yank?: string | boolean;
   noBrowser?: boolean;
 };
 
 export async function exec(
   denops: Denops,
   commitish: string,
-  options: ExecOptions = { yank: false },
+  options: ExecOptions = {},
 ): Promise<void> {
   const [verbose, aliases] = await batch.collect(denops, (denops) => [
     option.verbose.get(denops),
@@ -56,7 +56,7 @@ export async function exec(
     aliases,
   });
 
-  if (options.yank !== false) {
+  if (options.yank != null && options.yank !== false) {
     await yank(
       denops,
       url.href,