LuaLS / LuaLS/lua-language-server
Override is prevented from linter
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Lua
- Estrelas
- 4.4k
- Forks
- 442
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
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
local a = {}
a.__index = a
function a:m1()
end
-- b inherit from a
local b = setmetatable({}, a)
-- override m2()
function b:m2() --- no warning generated
end
Actual Behaviour
local a = {}
a.__index = a
function a:m1()
end
-- b inherit from a
local b = setmetatable({}, a)
-- override m2()
function b:m2() --- duplicate field `m`
end
Reproduction steps
- Just copy the code and lint them
Additional Notes
Inheritance and override methods is prevent by LuaLS linter.
Override is a common and basic concept. But LuaLS seems can't recognize it but provide warnings.
In the case, m2 is not a duplicate field, but just shadowing or overriding.
Log File
No response
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Reproduza o diagnóstico na Visual Studio Code Extension usando o snippet de Lua em Diagnostics/Syntax Checking. Em seguida, rastreie o diagnóstico do linter para campos duplicados e seu tratamento da herança de setmetatable; a tarefa estará concluída quando m2 não for reportado como duplicado, enquanto os campos duplicados não relacionados continuarem sendo reportados.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- lua
- Domínio
- devtools
- Tipo de issue
- Bug
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100