Skip to content

Conversation

@xiaoshihou514
Copy link
Member

No description provided.

@xiaoshihou514 xiaoshihou514 merged commit 55bbf64 into nvimdev:main May 16, 2025
3 checks passed
@glepnir
Copy link
Member

glepnir commented May 23, 2025

emm this will cause a ignore pattern buffer still have BufWritePre event reigseter . this is not correct.

@xiaoshihou514
Copy link
Member Author

emm this will cause a ignore pattern buffer still have BufWritePre event reigseter . this is not correct.

The previous logic would not attatch if any of the formatters ignored this buffer, which is not correct either.

@glepnir
Copy link
Member

glepnir commented May 23, 2025

previous logic does not register this event for buffer 2. but now registered

image

@xiaoshihou514
Copy link
Member Author

previous logic does not register this event for buffer 2. but now registered

image

Yes, I removed that logic because it was incorrect.

@glepnir
Copy link
Member

glepnir commented May 24, 2025

reason ?????? this is a bug which cause the server can not sync with increment

I got what you mean

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