GDQuest / GDQuest/GDScript-formatter

Reorder code: Add support for attaching some warnings to the line above them

オープン
#164 コメント 6 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
Rust
スター
455
フォーク
39
平均マージ
1日 15時間
マージ済み PR(30日)
4

説明

Using gdscript-formatter 0.14.0 via AUR
with Helix 25.07.1 with `formatter = { command = "gdscript-formatter", args = ["--reorder-code"] }` and `auto-format = true`

Dunno if it is related or was fixed with cf1c680.

Before:
```
extends Node

@warning_ignore_start("unused_signal")
signal signal1
signal signal2
@warning_ignore_restore("unused_signal")

func f1() -> void:
print("Hi!")
```

After:
```
extends Node

@warning_ignore_start("unused_signal")
signal signal1
signal signal2

@warning_ignore_restore("unused_signal")
func f1() -> void:
print("Hi!")
```

Expected behavior:

@warning_ignore_restore should stay at the same line.

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

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

issue に示されている警告ディレクティブと --reorder-code オプションを使って、gdscript-formatter でフォーマットの変更を再現します。reorder-code のエントリーポイントを追跡し、@warning_ignore_restore が後続の関数に対してどの位置に配置されるかを確認します。restore ディレクティブが元の行に残り、余分な空行が入らなければ完了です。

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

評価

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

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

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