String Templates Formatting for """ Multi-Line is "not nice"
まだ誰も着手していません。
評価
調査の方向性
web/ui/src/main/java/dev/enola/web/ui/ThingUI.java の複数行 string-template の例から始め、#981 で導入された JEP-430 のサポートを確認します。複数行テンプレートに対する formatter の動作を追跡し、目的の補間レイアウトを示す回帰テストを追加して、フォーマット後の出力を検証します。
索引モデルが issue の本文から書いたものです。
説明
#981 introduced JEP-430 / JEP-459 String Templates support.
This works great (~leaves it alone) for "single line", e.g. var html = STR."<table class=\"\{cssClass}\">";
However when exploring using this for """ Multi-Line it currently formats like this:
return STR."""
<table class="\{
s(cssClass)}">
<tbody>
</tbody>
</table>
""";
To my eyes, that line break (as if it was a Builder type thing, presumably?) looks fairly "ugly"; I would expect it to simply look like this instead:
return STR."""
<table class="\{s(cssClass)}">
<tbody>
</tbody>
</table>
""";
Do folks agree?
@cushon
- 主要言語
- Java
- スター
- 6.2k
- フォーク
- 936
- 平均マージ
- 6分
- マージ済み PR(30日)
- 3
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
google/google-java-format のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
google/google-java-format#1094 · コメント 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
google/google-java-format#1450 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
google/google-java-format#1439 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 62/100
google/google-java-format#1436 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
google/google-java-format#1428 · コメント 4 件 ·
google/google-java-format の issue をすべて見る
似ている issue
-
area/plugin
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
kestra-io/plugin-kestra#190 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
apache/rocketmq-dashboard#5064 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
wso2/dpdp-accelerator#287 ·