LuaLS / LuaLS/lua-language-server

Ability to use specific enum keys as params

Offen
#2,274 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

It would be nice to be able to use specific enum keys as params. For example today if you try to do something like this:

---@enum TestEnum
TestEnum = {
	ONE = 1,
	TWO = 2,
	THREE = 3,
}

---@param test TestEnum.ONE
local function test(test)
end

test(TestEnum.ONE)
test(1)

You will get an error:

Cannot assign `integer` to parameter `TestEnum.ONE`.
- `integer` cannot match `TestEnum.ONE`
- Type `integer` cannot match `TestEnum.ONE`
- Type `number` cannot match `TestEnum.ONE`

Interestingly lls understands that TestEnum.ONE is a real thing just not that it is equal to TestEnum.ONE or 1. For exmaple if you try this:

---@enum TestEnum
TestEnum = {
	ONE = 1,
	TWO = 2,
	THREE = 3,
}

---@param test TestEnum.One
local function test(test)
end

test(TestEnum.ONE)
test(1)

You will get an error:

---@param test TestEnum.NonEnumValue

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

Da im Issue keine Datei, kein Test und kein Einstiegspunkt angegeben ist, beginne damit, die Prüfung von Enum-Typen und die Behandlung der Zuweisbarkeit von Parametern in lua-language-server zu lokalisieren. Als erledigt gilt die Aufgabe, wenn TestEnum.ONE für TestEnum.ONE akzeptiert wird und der zugrunde liegende Wert 1 konsistent behandelt wird, während ungültige Enum-Schlüssel weiterhin abgelehnt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
devtools
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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