nodejs は sass と JavaScript ライブラリのビルドでしか使用しないため分離する

[nanasess]

概要(Overview・Refs Issue)

• nodejs は sass と JavaScript ライブラリのビルドでしか使用しないため、コンテナを分離する
• node12 は既にEOLになっているため、 docker build で警告が出る

方針(Policy)

• docker-compose.nodejs.yml を追加し、 nodejs は別コンテナで実行する

実装に関する補足(Appendix)

以下のようにしてビルド可能

# 初回およびpackage-lock.jsonに変更があったとき
docker compose -f docker-compose.yml -f docker-compose.dev.yml -f docker-compose.nodejs.yml run --rm -T nodejs npm ci
# Sass のビルド
docker compose -f docker-compose.yml -f docker-compose.dev.yml -f docker-compose.nodejs.yml run --rm -T nodejs npm run build
# JavaScript ライブラリのみのビルド
docker compose -f docker-compose.yml -f docker-compose.dev.yml -f docker-compose.nodejs.yml run --rm -T nodejs npx webpack

テスト（Test)

相談（Discussion）

適宜マージ先を 4.3 ブランチへ変更お願いいたします

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更はありません
• フックポイントの呼び出しタイミングの変更はありません
• フックポイントのパラメータの削除・データ型の変更はありません
• twigファイルに渡しているパラメータの削除・データ型の変更はありません
• Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
• 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cae3bf9) 82.56% compared to head (4169b0d) 82.56%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff            @@
##                4.2    #6075   +/-   ##
=========================================
  Coverage     82.56%   82.56%           
  Complexity     6436     6436           
=========================================
  Files           477      477           
  Lines         25884    25884           
=========================================
+ Hits          21371    21372    +1     
+ Misses         4513     4512    -1     

Flag	Coverage Δ
E2E	69.49% <ø> (ø)
Unit	79.76% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dotani1111]

@nanasess
PRありがとうございます。 👍
現行のJSライブラリとの差異が発生してないか確認を進めます。