redhat-developer / redhat-developer/vscode-java

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

オープン
#3,326 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
TypeScript
スター
2.3k
フォーク
546
平均マージ
20時間 1分
マージ済み PR(30日)
11

説明

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

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、vscode-javaコードベースでsource.generate.finalModifiersの実装とテストを見つけます。issueのUser.ageの例のような、アノテーションが付けられた可変のJavaフィールドで動作を再現し、その後、フィールドが外部から変更可能な場合に設定がfinalを追加しないことを示す回帰テストを追加します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java, typescript, vscode
領域
developer-experience, tooling
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。