Skip to content

conform: respect vim.lsp.formatOnSave and vim.lsp.mappings.toggleFormatOnSave - #891

Merged
NotAShelf merged 5 commits into
NotAShelf:mainfrom
horriblename:conform-format-on-save-fix-v2
May 9, 2025
Merged

conform: respect vim.lsp.formatOnSave and vim.lsp.mappings.toggleFormatOnSave#891
NotAShelf merged 5 commits into
NotAShelf:mainfrom
horriblename:conform-format-on-save-fix-v2

Conversation

@horriblename

Copy link
Copy Markdown
Collaborator

supersedes #768

@horriblename horriblename changed the title Conform format on save fix v2 conform: respect vim.lsp.formatOnSave May 9, 2025
github-actions Bot pushed a commit that referenced this pull request May 9, 2025
@horriblename horriblename changed the title conform: respect vim.lsp.formatOnSave conform: respect vim.lsp.formatOnSave and vim.lsp.mappings.toggleFormatOnSave May 9, 2025
@github-actions

github-actions Bot commented May 9, 2025

Copy link
Copy Markdown

🚀 Live preview deployed from 65e1fdd

View it here:

Debug Information

Triggered by: NotAShelf

HEAD at: conform-format-on-save-fix-v2

Reruns: 828

@NotAShelf NotAShelf left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Could you add a quick CHANGELOG entry for this? Not a breaking change, but should be communicated more clearly imo.

@horriblename
horriblename marked this pull request as draft May 9, 2025 02:10
@horriblename
horriblename force-pushed the conform-format-on-save-fix-v2 branch from fb00f60 to 9b505ab Compare May 9, 2025 02:12
@horriblename
horriblename force-pushed the conform-format-on-save-fix-v2 branch from 9b505ab to 955f204 Compare May 9, 2025 02:13
@horriblename

Copy link
Copy Markdown
Collaborator Author

by CHANGELOG you mean release notes right?

@horriblename
horriblename marked this pull request as ready for review May 9, 2025 02:17
@NotAShelf

Copy link
Copy Markdown
Owner

Yes, sorry I'm a little tired. Just a regular entry as we usually add them.

github-actions Bot pushed a commit that referenced this pull request May 9, 2025
@NotAShelf
NotAShelf merged commit 65e1fdd into NotAShelf:main May 9, 2025
@github-actions

github-actions Bot commented May 9, 2025

Copy link
Copy Markdown

✅ Preview has been deleted successfully!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants