Skip to content

TZ=UTC のテストスクリプトを cross-env 経由にしてWindowsで動くようにする#6406

Merged
TinyKitten merged 1 commit into
devfrom
chore/cross-env-tz-utc
Jul 12, 2026
Merged

TZ=UTC のテストスクリプトを cross-env 経由にしてWindowsで動くようにする#6406
TinyKitten merged 1 commit into
devfrom
chore/cross-env-tz-utc

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 12, 2026

Copy link
Copy Markdown
Member

概要

npm test / npm run watch:testTZ=UTC 環境変数指定が Windows の PowerShell / cmd では解釈されずコマンドが失敗していたため、cross-env 経由に変更してクロスプラットフォームで動くようにしました。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • package.jsontestTZ=UTC jestcross-env TZ=UTC jest に変更
  • package.jsonwatch:testTZ=UTC jest --watchcross-env TZ=UTC jest --watch に変更
  • cross-env^10.1.0)は既に devDependencies に存在し、android / ios スクリプトでも利用済みのため依存追加は不要

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

関連Issue

スクリーンショット(任意)

Summary by CodeRabbit

  • バグ修正
    • テスト実行時のタイムゾーン設定を改善し、環境に依存せず安定してテストを実行できるようにしました。
    • 通常のテスト実行とウォッチモードの両方に対応しています。

@TinyKitten TinyKitten self-assigned this Jul 12, 2026
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4da67aff-04e5-42f1-b01c-5b0826d88e14

📥 Commits

Reviewing files that changed from the base of the PR and between b91a4b8 and 35914c2.

📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

テストおよびウォッチモードのスクリプトで、TZ=UTC の指定を cross-env 経由に変更しました。typecheck は変更していません。

Changes

テスト環境設定

Layer / File(s) Summary
テストスクリプトの環境変数設定
package.json
testwatch:testcross-env TZ=UTC を使用するよう更新され、typechecktsc --noEmit のままです。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

ぴょんと跳ねるウサギは言う
UTCの時計をそろえよう
テストも監視も軽やかに
cross-envで駆け抜ける
にんじん色の緑の合図

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PRタイトルは変更内容の要点を簡潔に示しており、Windows対応の意図も明確です。
Description check ✅ Passed テンプレートの各見出しを概ね満たしており、概要・変更内容・テスト方針が具体的に書かれています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/cross-env-tz-utc

Comment @coderabbitai help to get the list of available commands.

@TinyKitten

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TinyKitten

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TinyKitten TinyKitten merged commit 8db439d into dev Jul 12, 2026
7 checks passed
@TinyKitten TinyKitten deleted the chore/cross-env-tz-utc branch July 12, 2026 21:01
@TinyKitten TinyKitten mentioned this pull request Jul 12, 2026
9 tasks
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