From da8fd5cf3e0ec4dd933910d92ca643c02b362b25 Mon Sep 17 00:00:00 2001 From: Fitrah Muhammad Date: Tue, 13 Feb 2024 21:34:00 +0800 Subject: [PATCH] fix(migrate): avoid ternary 'falls through' state See: https://riptutorial.com/lua/example/26503/emulating-ternary-operator-with--and---or--logical-operators-#undefined --- lua/rose-pine/config.lua | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lua/rose-pine/config.lua b/lua/rose-pine/config.lua index 7b6f8aa..562d2a4 100644 --- a/lua/rose-pine/config.lua +++ b/lua/rose-pine/config.lua @@ -133,11 +133,11 @@ local function migrate(options) -- These never actually existed, but may be set intuitively by the user -- because of `disable_italics` existing. - options.styles.bold = (options.disable_bold or options.disable_bolds) and false or options.styles.bold + options.styles.bold = not (options.disable_bold or options.disable_bolds) and options.styles.bold or false -- Similar to bold options, `disable_italic` never existed but could be a -- common typo of the actual `disable_italics`. - options.styles.italic = (options.disable_italic or options.disable_italics) and false or options.styles.italic + options.styles.italic = not (options.disable_italic or options.disable_italics) and options.styles.italic or false -- Set h1 through h6 to the same color if only one is specified if type(options.groups.headings) == "string" then