Skip to content

[jaspr_cli] Ensure settings are preserved in updateSetting call#761

Merged
schultek merged 1 commit intoschultek:mainfrom
parlough:fix/cli-ensure-settings-are-preserved
Mar 8, 2026
Merged

[jaspr_cli] Ensure settings are preserved in updateSetting call#761
schultek merged 1 commit intoschultek:mainfrom
parlough:fix/cli-ensure-settings-are-preserved

Conversation

@parlough
Copy link
Copy Markdown
Contributor

@parlough parlough commented Mar 1, 2026

Before this, updateSetting didn't preserve updates if the settings file didn't exist originally, since (_settings ?? {})[key] = value; didn't save the created map.

@parlough parlough requested a review from schultek as a code owner March 1, 2026 05:11
@docs-page
Copy link
Copy Markdown

docs-page Bot commented Mar 1, 2026

To view this pull requests documentation preview, visit the following URL:

docs.page/schultek/jaspr~761

Documentation is deployed and generated using docs.page.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2026

Package Version Report

No packages have been updated.

Copy link
Copy Markdown
Owner

@schultek schultek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks

@schultek schultek merged commit 0cca31e into schultek:main Mar 8, 2026
11 checks passed
@parlough parlough deleted the fix/cli-ensure-settings-are-preserved branch March 9, 2026 07:29
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.

2 participants