Skip to content

consoleの実装とテストをpackages/jsEval/に移動#226

Open
tknkaa wants to merge 1 commit into
mainfrom
refactor/move-console-to-jseval
Open

consoleの実装とテストをpackages/jsEval/に移動#226
tknkaa wants to merge 1 commit into
mainfrom
refactor/move-console-to-jseval

Conversation

@tknkaa

@tknkaa tknkaa commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

JavaScript だけのテストを REPL の共通テストファイルに書きたくない、という話があってのリファクタ PR です。
JavaScript の console のロジックを createReplConsole という関数に切り出し、@my-code/js-eval ワークスペースに移動しました。それに伴い、@my-code/runtime にあった JavaScript だけのテストも @my-code/js-eval に移動しました。

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
my-code 5beebc9 Commit Preview URL

Branch Preview URL
Jul 24 2026, 01:15 PM

@tknkaa
tknkaa marked this pull request as draft July 24, 2026 01:55
replのテストファイルにJS専用のconsole.time/timeEndテストが混在していたため、
consoleの実装(createReplConsole)をjsEval側に切り出し、対応するテストも移動した。

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@tknkaa
tknkaa force-pushed the refactor/move-console-to-jseval branch from 2160682 to 5beebc9 Compare July 24, 2026 13:10
@tknkaa
tknkaa marked this pull request as ready for review July 24, 2026 13:37
@tknkaa
tknkaa requested a review from na-trium-144 July 24, 2026 13:42
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.

1 participant