-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Formatting and linting requires CocRestart
+ saving twice to start working
#13
Comments
Try v1.4.1 |
v1.4.1 still needs me to I think I might need to create an MVP for this, but need more time. |
Can't reproduce, linting and formatting works on saving with Does |
Add |
Before restartOutput
No After restartAfter I ran
|
I will look when I get back in a few hours, I missed this
…On Tue, Jan 2, 2024, 3:26 AM Heyward Fann ***@***.***> wrote:
Can't reproduce, linting and formatting works on saving with w.
Does :CocCommand editor.action.formatDocument work?
—
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAGNYERL4OXL4VGW3QG2GDYMPHGFAVCNFSM6AAAAABBIIPV6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZTG44DEMRVGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Still can't reproduce, after Tested with your |
It may be my system, or vim configuration, then. With
|
29.movYou can set |
@fannheyward It depends on where
webm: 2024-01-07.09-26-23.-.coc-biome.-.project.root.vs.package.webm
|
Reproduced. It should be a problem caused by the workspace root detecting. When you're in Set |
That works, but now the 2024-01-14.07-52-00.-.coc-biome.-.biome.with.requireConfiguration.mp4
From a technical perspective this makes sense. Is that something to expect for language servers? For coc? coc plugins? Are their exceptions to this? As a user I'd hope for the behavior to be the same as |
The key issue is workspace root detecting.
coc uses Why coc uses this as project root? After you open coc-biome can't search upwards to use biome.json in parent folders. Why |
@fannheyward Thank you for your help. I don't see any indication of I am closing this, but if there's any disagreement, let me know! 🙏 |
May be related to #11, but in biome 1.4.0 and vim 9.0.
Workaround (For each vim instance)
Works with #12
App.tsx
:CocRestart
:w
(no formatting yet):w
The text was updated successfully, but these errors were encountered: