Empty line before comment should (sometimes) be preserved
まだ誰も着手していません。
評価
調査の方向性
zeek-format によるマイナーコメント前の空行の扱いから始め、完了したイベントの後にあるコメントと、{ の後の最初のノードであるコメントを比較します。例をフォーマットしたときに空行が保持され、結果をもう一度フォーマットしても変更されないことを確認します。一方で、{ の直後の空行は引き続き削除されることも確認します。
索引モデルが issue の本文から書いたものです。
説明
Currently zeek-format removes empty lines before any minor comments, e.g.,
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line before.
first gets formatted to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
and then again (breaking idempotency) to
event new_event(name: string, params: call_argument_vector)
{
print name;
}
# Should preserve empty line.
I would have expect this to not change formatting at all.
zeek-format should still remove empty after e.g., { (comment first node in block).
Labelling as bug due to idempotency breaking.
- 主要言語
- Python
- スター
- 11
- フォーク
- 4
- 平均マージ
- 3日 19時間
- マージ済み PR（30日）
- 1
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
zeek/zeekscript のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
zeek/zeekscript#137 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
zeek/zeekscript#120 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
zeek/zeekscript#83 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
zeek/zeekscript#69 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
zeek/zeekscript#63 ·
zeek/zeekscript の issue をすべて見る
似ている issue
-
agent-ready documentation needs-triage
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
-
documentation
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" オープン
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
-
instance instance add
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
searxng/searx-instances#939 · コメント 1 件 ·
-
area-deployment area-integrations triage:bot-seen
難易度 2/5 半日 初心者へのやさしさ 86/100