LuaLS / LuaLS/lua-language-server

Better global recognizing for loadfile

Offen
#143 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement feat/LuaCats Annotations
Vorherrschende Sprache
Lua
Sterne
4.4k
Forks
442
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

In some cases you might need to make the plugin recognize a specific local variable as global, without declaring that variable as an actual global, one of these cases are using load and loadfile were you can pass an env variable that represents the chunk environment.

*mainFile.lua

local env = setmetatable(
  {foo = function(a, b) return a == b end},
  {__index = _G}
)

loadfile('./secondFile.lua', 't', env)

*secondFile.lua

foo(1, 2) -- Undefined global `foo` . Lua Diagnostics.(undefined-global)

So i would suggest better recognizing for load/loadfile/loadstring environment or/and a custom emmylua annotations --- @global that marks the local var as a global one.

Thanks in advance.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit nachzuverfolgen, wie der Language Server das Diagnoseereignis undefined-global meldet und wie er Umgebungen für load, loadfile und loadstring behandelt. Vergleiche dieses Verhalten mit dem Beispiel mainFile.lua und secondFile.lua, und ermittle anschließend, ob die Erkennung der Umgebung oder eine @global-Annotation der vorgesehene Umfang ist. Als abgeschlossen gilt die Aufgabe, wenn das Beispiel foo unter dem gewählten Verhalten nicht mehr als undefiniert meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
devtools
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.