Skip to content
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

formatOnSave & autocomplete became slower after update with Deno 1.37.0 #931

Closed
ynwd opened this issue Sep 21, 2023 · 8 comments
Closed

Comments

@ynwd
Copy link

ynwd commented Sep 21, 2023

Describe the bug

  • formatOnSave and autocomplete became slower after update with Deno 1.37.0
  • this did not happen in previous versions

what actually happened?

Operating System

  • Mac
@ynwd ynwd changed the title formatOnSave became slower after update with Deno 1.37.0 formatOnSave & autocomplete became slower after update with Deno 1.37.0 Sep 22, 2023
@ynwd
Copy link
Author

ynwd commented Sep 22, 2023

For a temporary solution, use esbenp.prettier-vscode again.

{
  "deno.enable": true,
  "deno.lint": true,
  "deno.codeLens.test": true,
  "editor.formatOnSave": true,
  "deno.importMap": "import_map.json",
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "[typescriptreact]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[typescript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[javascriptreact]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[javascript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[css]": {
    "editor.defaultFormatter": "vscode.css-language-features"
  }
}

@bartlomieju
Copy link
Member

CC @nayeemrmn can you take a look?

@ynwd
Copy link
Author

ynwd commented Sep 23, 2023

For a temporary solution, use esbenp.prettier-vscode again.

{
  "deno.enable": true,
  "deno.lint": true,
  "deno.codeLens.test": true,
  "editor.formatOnSave": true,
  "deno.importMap": "import_map.json",
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "[typescriptreact]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[typescript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[javascriptreact]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[javascript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[css]": {
    "editor.defaultFormatter": "vscode.css-language-features"
  }
}

It not works, btw.

So i downgrade to 1.36.4 -- and the problem is gone.

@bartlomieju @nayeemrmn

@nayeemrmn
Copy link
Collaborator

@ynwd If the problem is still there when changing the default formatter, it's not the formatter that's slow. Is it just the LSP in general being unresponsive? I can't see anything like that. Can you give a more detailed reproduction e.g. a sample file in a new project?

@ynwd
Copy link
Author

ynwd commented Sep 24, 2023

@ynwd If the problem is still there when changing the default formatter, it's not the formatter that's slow. Is it just the LSP in general being unresponsive? I can't see anything like that. Can you give a more detailed reproduction e.g. a sample file in a new project?

@nayeemrmn
very strange. after re-downloading 1.37.0, the problem went away. I will reopen this issue if the problem occurs again. Thank You.

@ynwd ynwd closed this as completed Sep 24, 2023
@nayeemrmn
Copy link
Collaborator

@ynwd Looks like this is the same as denoland/deno#20136. Watch that issue for updates.

@ynwd
Copy link
Author

ynwd commented Sep 24, 2023

on a very small project (like hello world http server), this problem is not found.

but when I switch to the bigger one (with watch mode and a complex import_map -- fullstack apps), this problem is found again.

so my temporary solution is back to use previous version (v1.36.4)

@ynwd ynwd reopened this Sep 24, 2023
@nayeemrmn
Copy link
Collaborator

Closing in favour of #943. Thanks for reporting

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

No branches or pull requests

3 participants