Skip to content

Add node.sh to /judge directory for consistency#72

Merged
toshi0806 merged 1 commit intomainfrom
add-node-sh-to-judge
Oct 30, 2025
Merged

Add node.sh to /judge directory for consistency#72
toshi0806 merged 1 commit intomainfrom
add-node-sh-to-judge

Conversation

@toshi0806
Copy link
Member

変更内容

JavaScript実行用のnode.shスクリプトを/judgeディレクトリに配置し、java.shcpp.shとの配置一貫性を確保しました。

追加ファイル

  • js/node.sh - Node.jsのスタックサイズ制御を行う実行ラッパー

変更箇所

  • Dockerfile (フル版): line 447-449に追加
  • Dockerfile.lite (ライト版): line 327-329に追加

両バージョンで以下のセクションを追加:

# Node.js execution script
COPY js/node.sh /judge/node.sh
RUN chmod +x /judge/node.sh

影響範囲

  • 破壊的変更なし: 新しいファイルの追加のみ
  • 既存機能への影響なし: 他の言語の実行には影響しません

関連Issue

Fixes #71

次のステップ

このPRマージ後:

  1. 新しいイメージをビルド・公開
  2. atcoder-env側でmakefileの参照先を/judge/node.shに更新(Move node.sh to /judge directory for consistency with other language wrappers atcoder-env#112

- Add js/node.sh script for JavaScript execution with stack size control
- Copy node.sh to /judge directory in both Dockerfile variants
- Makes JavaScript wrapper consistent with java.sh and cpp.sh placement

Fixes #71
@toshi0806 toshi0806 merged commit 4156a76 into main Oct 30, 2025
5 checks passed
@toshi0806 toshi0806 deleted the add-node-sh-to-judge branch October 30, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add node.sh to /judge directory for consistency

1 participant