fix(denols): remove unneeded lsp attach

[sigmaSd]

fix #1998

I don't have a good idea how does this fixes the issue, but so far in my testing with this pr it seems to work perfectly.

[sigmaSd]

The commit that added this #753

@kitagry can you confirm that this is not needed anymore?

[kitagry]

I added this code in order to use lsp functions (e.x. goto definition) in virtual text file. And, is it work correctly?

I'll see it this night. Thank you.

[sigmaSd]

Yes it seem to work inside virtual document without it.

[justinmk]

lint CI is failing because of wrong indentation

[kitagry]

Hmm, I can't reproduce this bug. Can you try with neovim develpment build?

NVIM v0.8.0-dev+1470-g0124a7bfa
Build type: Release
LuaJIT 2.1.0-beta3
nvim-lspconfig c55e830

Yes it seem to work inside virtual document without it.

I think it works only when you specify single_file_support=true.

[sigmaSd]

I think it works only when you specify single_file_support=true.

I can confirm this, do you have a suggestion on how to proceed then? is there a disadvantage of making single_file_support=true by default?

[kitagry]

is there a disadvantage of making single_file_support=true by default?

I think there is no disadvantage. I think you should this PR changes with single_file_support=true by default.