LuaLS / LuaLS/lua-language-server
Override is prevented from linter
Nessuno ha ancora preso questa issue.
- Lingua principale
- Lua
- Stelle
- 4.4k
- Fork
- 442
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Riproduci la diagnostica nella Visual Studio Code Extension usando lo snippet Lua sotto Diagnostics/Syntax Checking. Traccia quindi la diagnostica del linter relativa ai campi duplicati e la gestione dell’ereditarietà di setmetatable; il lavoro è completato quando m2 non viene segnalato come duplicato, mentre i campi duplicati non correlati continuano a esserlo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- lua
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100