redhat-developer / redhat-developer/vscode-java

source.generate.finalModifiers should not add final modifiers to the fields

Offen
#3,326 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
TypeScript
Sterne
2.3k
Forks
546
Ø Merge
20 Std. 1 Min.
Gemergte PRs (30 T.)
11

Beschreibung

If I have a field line that:

class User {
    @Parameter(names = { "--age", "-a" }, required = true, description = "User age")
    public int age = 1;
}

following option:

    "[java]": {
        "editor.codeActionsOnSave": {
            "source.generate.finalModifiers": true
        }
    }

add final modifier to it:

class User {
    @Parameter(names = { "--age", "-a" }, required = true, description = "User age")
    public final int age = 1;
}

but it should not, because it can be changes from other code or library like JCommander.

Environment
  • Operating System: macOS Sonoma 14.0
  • JDK version: Zulu 8.72.0.17-CA-macos-aarch64
  • Visual Studio Code version: 1.82.2
  • Java extension version: 0.25.14

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

Beginne damit, die Implementierung und die Tests für source.generate.finalModifiers in der vscode-java-Codebasis zu finden. Reproduziere das Verhalten mit einem annotierten, veränderlichen Java-Feld wie im User.age-Beispiel des Issues und füge anschließend eine Regressionstestabdeckung hinzu, die zeigt, dass die Einstellung kein final hinzufügt, wenn das Feld extern geändert werden kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java, typescript, vscode
Bereich
developer-experience, tooling
Issue-Typ
Bug
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.