LuaLS / LuaLS/lua-language-server
Global in one file affects diagnostics in another
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Lua
- Estrellas
- 4.4k
- Forks
- 442
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Windows
What is the issue affecting?
Diagnostics/Syntax Checking
Expected Behaviour
If I have 1.lua:
function Main()
foobar = 1
end
and 2.lua:
function Main()
print(foobar)
end
in the same workspace/directory.
I would expect 2.lua to show foobar as undefined (unless I explicitly require 1.lua)
Actual Behaviour
No error is shown.
Additional Notes
This might be a pecularity of how I'm using lua (each file defines a separate plugin) but I would be surprised if this is an uncommon usage pattern. At the very least there should be a way to configure the behaviour.
Also - I'd be curious to understand the thinking behind the existing behaviour?
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el ejemplo en la Visual Studio Code extension con 1.lua y 2.lua en el mismo workspace y, después, rastrea cómo el language server comparte las variables globales entre archivos. Determina cómo debería afectar el uso explícito de require a los diagnósticos y si se necesita una opción de configuración. El trabajo estará terminado cuando 2.lua informe de que foobar no está definida, a menos que 1.lua se requiera explícitamente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- lua, vscode
- Área
- devtools
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 30/100