LuaLS / LuaLS/lua-language-server

[FEATURE-REQUEST] Configuration option to disable didRenameFiles prompt

Offen
#2,789 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

When the language server is notified that a file rename has occured you are always prompted with "Do you want to modify the require path?"

I find that to be bothersome and would prefer if the refactoring would happen silently.
As far as I can tell there's currently no configuration option for this.

The responsible code seems to be in lua-language-server/script/core/modifyRequirePath.lua:67:

    local _, index = client.awaitRequestMessage('Info', lang.script.WINDOW_MODIFY_REQUIRE_PATH, {
        lang.script.WINDOW_MODIFY_REQUIRE_OK
    })

Would it be possible to simply add a check for a config option here?

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

Beginnen Sie bei lua-language-server/script/core/modifyRequirePath.lua:67 und verfolgen Sie, wie Konfigurationsoptionen definiert und gelesen werden. Fügen Sie eine Konfigurationsoption für den didRenameFiles-Prompt hinzu und überprüfen Sie anschließend, dass die Deaktivierung dieser Option das Refactoring stillschweigend fortsetzen lässt, während der bestehende Prompt standardmäßig verfügbar bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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