LuaLS / LuaLS/lua-language-server
Ability to use specific enum keys as params
Dieses Issue hat noch niemand übernommen.
- 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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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