redhat-developer / redhat-developer/vscode-java

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

Ouverte
#3,326 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
TypeScript
Étoiles
2.3k
Forks
546
Merge moyen
20 h 1 min
PR mergées (30 j)
11

Description

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser l’implémentation et les tests de source.generate.finalModifiers dans la base de code de vscode-java. Reproduisez le comportement avec un champ Java annoté et mutable comme dans l’exemple User.age de l’issue, puis ajoutez une couverture de régression montrant que le paramètre n’ajoute pas final lorsque le champ peut être modifié de l’extérieur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java, typescript, vscode
Domaine
developer-experience, tooling
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.