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

isIgnoredFiles with default constructed path due to drive letter #752

Open
MohamedBannona opened this issue Aug 22, 2024 · 3 comments · May be fixed by #819
Open

isIgnoredFiles with default constructed path due to drive letter #752

MohamedBannona opened this issue Aug 22, 2024 · 3 comments · May be fixed by #819
Labels
bug Something isn't working

Comments

@MohamedBannona
Copy link
Contributor

I'm not sure this happens cause of the sourcemap, but I did some tests and I only get this error in scripts included in the sourcemap, maybe because I'm using argon.
image

@JohnnyMorganz
Copy link
Owner

In the next release I'm going to improve the error message for this. Can you show me the updated error message when that happens?

@JohnnyMorganz JohnnyMorganz added the bug Something isn't working label Sep 11, 2024
@MohamedBannona
Copy link
Contributor Author

D:/Mopr/test is project root
image

LSP[luau-lsp]: Error INVALID_SERVER_MESSAGE: {
  error = {
    code = -32603,
    message = "isIgnoredFile failed: relative path is default-constructed when constructing d:/Mopr/test/src/Server/Main.server.luau against D:/Mopr/test"
  },
  jsonrpc = "2.0"
}

@JohnnyMorganz
Copy link
Owner

Ah the age-old problem of case sensitivity of the drive letter. It seems that Neovim uses a capital letter while VSCode uses lowercase (for some historic reasons apparently)

We need to normalise the case letter here.

@JohnnyMorganz JohnnyMorganz changed the title [email protected] keeps errors when editing scripts included in sourcemap isIgnoredFiles with default constructed path due to drive letter Sep 19, 2024
miwwow added a commit to miwwow/nvim that referenced this issue Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants