LuaLS / LuaLS/lua-language-server
Override is prevented from linter
Personne n'a encore pris cette issue.
- Langage dominant
- Lua
- Étoiles
- 4.4k
- Forks
- 442
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Reproduisez le diagnostic dans la Visual Studio Code Extension à l’aide de l’extrait Lua sous Diagnostics/Syntax Checking. Suivez ensuite le diagnostic du linter concernant les champs en double et sa gestion de l’héritage de setmetatable ; la tâche est terminée lorsque m2 n’est pas signalé comme un doublon, tandis que les champs en double non liés continuent de l’être.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- lua
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100