Skip to content

empty cli or config args reset the value to the default - #1368

Merged
mitchellh merged 1 commit into
mainfrom
config-default
Jan 24, 2024
Merged

mitchellh merged 1 commit into
mainfrom
config-default

Conversation

@mitchellh

Copy link
Copy Markdown
Contributor

Fixes #1367

We previously special-cased optionals but we should do better and have this reset ANY type to the defined default value on the struct.

Fixes #1367

We previously special-cased optionals but we should do better and have
this reset ANY type to the defined default value on the struct.
@mitchellh
mitchellh merged commit 3beae1a into main Jan 24, 2024
@mitchellh
mitchellh deleted the config-default branch January 24, 2024 03:08
pluiedev pushed a commit that referenced this pull request May 14, 2026
After #1368, `command-palette-entry=` will no longer clear the entries like the documentation says. Since i couldn't find an existing issue or discussion about this, I assume no one is actually using it. So I put 1.4.0 here, lemme know if you want to change it to 1.3.2.

> I basically copied the `keybind` parsing code and doc.
pluiedev added a commit that referenced this pull request May 14, 2026
After #1368, `command-palette-entry=` will no longer clear the entries
like the documentation says. Since i couldn't find an existing issue or
discussion about this, I assume no one is actually using it. So I put
1.4.0 here, lemme know if you want to change it to 1.3.2.

> I basically copied the `keybind` parsing code and doc.
xi-guan pushed a commit to xi-guan/cmux-ghostty that referenced this pull request May 18, 2026
After ghostty-org#1368, `command-palette-entry=` will no longer clear the entries like the documentation says. Since i couldn't find an existing issue or discussion about this, I assume no one is actually using it. So I put 1.4.0 here, lemme know if you want to change it to 1.3.2.

> I basically copied the `keybind` parsing code and doc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Explicit default value in config for font-* variants causes configuration error

1 participant