File/folder renames where only capitalization changes causes assumed duplication
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 55/100
Piste de recherche
Start by tracing how the language server tracks an open file when its path changes only in capitalization, using the reproduction steps with a.lua renamed to A.lua on Windows. The report does not name a source file or test, so locate the file tracking and duplicate-field diagnostic entry points, then add a regression test for the rename case. Done when the class fields are no longer reported as duplicates after the rename.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
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
When a file or folder is renamed from, for example, test to Test, the language server should not suddenly mark everything as Duplicate-Defined-Field/etc.
Actual Behaviour
All fields within classes are marked as duplicates.
Reproduction steps
- Create a file,
a.lua. - Open the file so it's an opened tab in the editor.
- Rename it to
A.lua. - Write a basic class definition into it.
---@class X
---@field Y string
local x = {}
function x.new() end
- Observe the bug.
Additional Notes
Windows treats paths containing different capitalization as the same file, unlike linux. a.lua in Windows is the same as A.lua. I believe there is an assumption somewhere that a.lua does not equal A.lua and it causes the language server to bug out.
Log File
I just want to report this then sleep. If nobody else is able to reproduce/get logs, I will do so tomorrow.
- Langage dominant
- Lua
- Étoiles
- 4.4k
- Forks
- 449
- Merge moyen
- 15 j 6 h
- PR mergées (30 j)
- 4
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire 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.
Autres issues de LuaLS/lua-language-server
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 86/100
LuaLS/lua-language-server#3481 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 87/100
LuaLS/lua-language-server#3480 ·
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
LuaLS/lua-language-server#1776 ·
-
`continue -- comment` rejected with `runtime.nonstandardSymbol=["continue"]`Peut-être pris @ybai08 l’a pris il y a 3 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
LuaLS/lua-language-server#3474 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
LuaLS/lua-language-server#3470 ·
Toutes les issues de LuaLS/lua-language-server
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
jangala-dev/devicecode-lua#344 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
luainkernel/lunatik#2018 ·
Les mainteneurs répondent en général sous 1 jour
-
Kill Ishamuhale for Ishamuhale's FangPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug discord guide:04-the-barrens locale:enUS triage v0.2.6
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100