kelektiv/node-cron

refactor runtime check & typings of exclusive parameters timeZone & utcOffset

Open

#704 opened on 2023年9月30日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)JavaScript (7,868 stars) (650 forks)batch import
good first issuehacktoberfesthelp wantedstatus:readytype:refactor

説明

Does it? First condition precludes the second

https://github.com/kelektiv/node-cron/blob/a7dee32091b91dcf465e67c32a57d66151b5a774/src/job.ts#L56 https://github.com/kelektiv/node-cron/blob/a7dee32091b91dcf465e67c32a57d66151b5a774/src/job.ts#L57

https://github.com/kelektiv/node-cron/blob/a7dee32091b91dcf465e67c32a57d66151b5a774/src/job.ts#L65

コントリビューターガイド

技術スタック
typescript
領域
backend
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
TypeScript basicsUnderstanding of cron
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75
調査方針
Review src/job.ts lines 56-57 and 65 in the node cron repository. The condition on line 56 (`if (timeZone)`) makes the subsequent check on line 57 (`if (utcOffset)`) unreachable when both are exclusive. Examine the TypeScript type definitions to ensure they correctly enforce exclusivity. No linked pull requests or comments indicate a resolution, so the approach is to refactor the runtime logic and update typings accordingly.