[EllaCoat]

概要

CI の linter を datapack-linter（DHP 3.4.19）から mcf-linter-spark v0.1.1 に替えた。宣言の生成（make-declares-mcf.mts）は、spark が同じ形で書き出す .cache/dls.json を読むので変えていない。

主な変更

lint の step の中身とナレッジの説明だけを替え、job 名 lint・workflow 名・ファイル名と宣言の生成の手順はそのまま残した。

• 必須チェックの lint で検査を続けるよう、workflow の lint の step を mcf-linter-spark check に替えた
• 宣言の生成が読むキャッシュを用意するよう、同じ step で --symbols .cache/dls.json を書き出す
• 毎回のビルドとデータの取得を省くよう、spark の実行ファイルとバニラのデータを actions/cache で残す
• spark では外しても error・warning が 0 件だったので、core:define_gamerule の除外を外した
• ナレッジの CI の説明が実際と合うよう、docs/knowledge/ci-and-generation.md の linter と除外の記述を直した

設計判断

spark の入れ方と、lint の設定の扱いを決めた。

• spark は cargo install --git で EllaCoat/mcf-linter-spark の tag v0.1.1 を入れる
• spark は .vscode/settings.json の datapack.lint.* を DHP と同じ意味で読むので、設定は変えていない

動作確認

手元の検査と、この branch の push の CI が通った。

• master の写しで spark は error・warning 0 件、宣言の生成も成功
• 生成した宣言ファイルは、今の Asset の master のものと中身が一致（並び順だけ違う）
• tag v0.1.1 で入れた spark で、この branch の push の lint 通過（spark のビルド 52 秒、検査 1 秒、宣言の生成も成功）

未対応・注意

merge の前に要る作業は無い。merge の後は検査の範囲と、Asset へ push する宣言ファイルの並び順が変わり、#2307 とは衝突する。

• datapack-linter と違い毎回すべてのファイルを検査するので、master に残った診断はどの PR の lint にも出る
• merge 後の master の最初の run で、Asset の declares.d.mcfunction が並び順だけ変わって push される
• Asset が datapack-linter のままの間に要るので、deploy の [regenerate cache] は残した（Asset は 👷 CI の linter を datapack-linter から mcf-linter-spark に替える Asset#2218 で付け替える）
• ✨ 文字幅計算Libを追加 #2307 も datapack-linter.yml を変えていて衝突するので、✨ 文字幅計算Libを追加 #2307 の除外の指定を spark の --ignore に移す必要がある