LuaLS / LuaLS/lua-language-server
discard-returns warning is not triggered in while/repeat loops
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?
Annotations
### Expected Behaviour
@nodiscard annotated functions should always trigger the `discard-return`s warning in any context.
### Actual Behaviour
`discard-returns` does not trigger in _while_ and _repeat_ loops.
### Reproduction steps
```lua
---@nodiscard
local function bar()
--@nodiscard annotated function
end
while true do
bar() --< this DOES NOT trigger no-discard
break
end
repeat
bar() --< this DOES NOT trigger no-discard
break
until true
bar() -- < this DOES trigger no-discard
for index, value in ipairs({}) do
bar() -- < this DOES trigger no-discard
end
```
### Additional Notes
_No response_
### 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 il problema nell’estensione Visual Studio Code usando lo snippet Lua del report, quindi traccia il modo in cui l’avviso discard-returns gestisce i cicli while e repeat rispetto alle chiamate dirette e ai cicli for. Il lavoro è completato quando le chiamate @nodiscard in entrambe le forme di ciclo attivano l’avviso in modo coerente con gli altri contesti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- lua
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100