Skip to content

fix: remove format_on_save: true settings from Zed settings to respect user settings#1838

Merged
webpro merged 1 commit into
webpro-nl:mainfrom
trueberryless:fix/respect-user-format-on-save-option
Jun 29, 2026
Merged

fix: remove format_on_save: true settings from Zed settings to respect user settings#1838
webpro merged 1 commit into
webpro-nl:mainfrom
trueberryless:fix/respect-user-format-on-save-option

Conversation

@trueberryless

Copy link
Copy Markdown
Contributor

Description

Thanks to @mootari for mentioning on Discord that the format_on_save option should ideally not be overridden in our projects Zed settings, so users can experience it in the way the want.

Since we'll probably add pre-commit hooks in #1837 anyway, it doesn't really matter if the user always formats when saving the files 👍

@pkg-pr-new

pkg-pr-new Bot commented Jun 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/knip@1838
npm i https://pkg.pr.new/@knip/language-server@1838
npm i https://pkg.pr.new/@knip/mcp@1838

commit: cff2ef8

@webpro
webpro merged commit dc2a640 into webpro-nl:main Jun 29, 2026
30 checks passed
@trueberryless
trueberryless deleted the fix/respect-user-format-on-save-option branch June 29, 2026 06:01
@webpro

webpro commented Jul 2, 2026

Copy link
Copy Markdown
Member

🚀 This pull request is included in v6.24.0. See Release 6.24.0 for release notes.

Using Knip in a commercial project? Please consider becoming a sponsor.

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