_init() does not wrap like other functions + unwanted formatting case for a function call chain

オープン
  • 何か問題がありますか？
#370 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
55/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
rust
領域
tooling

調査の方向性

Reproduce both cases with gdscript-formatter 0.27.0 using the default max_line_length of 100, then inspect the formatter paths for function signatures and chained calls. Add regression coverage for both examples; done means formatting preserves already-valid input or produces output that stays within the configured limit.

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

説明

With 0.27.0 and the default settings (max_line_length: 100, tabs), the formatter produces lines longer than 100 columns, which gdscript-formatter lint then refuses. In both cases below, the input already passes lint and the formatted output doesn't, so a project that runs --check and lint together can't satisfy both.

1. A wrapped function signature is re-joined into one line of 101–109 columns

This looks like #300 again. That issue's own example still reproduces on 0.27.0 (it stays on one 108-column line).

Input (passes lint):

const SHADER := "res://a.gdshader"
const FADE_S := 1.0


func _init(
	node_name: String = "SkidMarks",
	shader_path: String = SHADER,
	fade_s: float = FADE_S,
) -> void:
	name = node_name

gdscript-formatter --stdout output:

func _init(node_name: String = "SkidMarks", shader_path: String = SHADER, fade_s: float = FADE_S) -> void:
	name = node_name

max-line-length:warning: Line is too long. Found 106 characters, maximum allowed is 100

Lengthening the last parameter shows that the signature stays on one line up to 109 columns and only wraps from 110.

2. A long argument to a chained call is pulled up onto a backslash-continued line

Input (passes lint, longest line 89 columns with tabs at 4):

func test_it() -> void:
	assert_that(x).override_failure_message(
		"this is a long failure message that goes on and on to push it beyond the limit %s" % x
	).is_true()

Output:

func test_it() -> void:
	assert_that(x) \
			.override_failure_message("this is a long failure message that goes on and on to push it beyond the limit %s"
	% x) \
			.is_true()

max-line-length:warning: Line is too long. Found 121 characters, maximum allowed is 100

This may be related to #360. Here, though, the input already fits, and the output both exceeds the limit and splits the % operand onto a line with less indentation.

Expected: in both cases, output that stays within max_line_length, or at least keeps an input that already fits.

macOS arm64, gdscript-formatter 0.27.0 release binary.

主要言語
Rust
スター
483
フォーク
40
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

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

GDQuest/GDScript-formatter のほかの issue

GDQuest/GDScript-formatter の issue をすべて見る

似ている issue

Rust の issue をもっと見る

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

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