kelektiv/node-cron

refactor runtime check & typings of exclusive parameters timeZone & utcOffset

Open

#704 建立於 2023年9月30日

在 GitHub 查看
 (3 留言) (1 反應) (0 負責人)JavaScript (7,868 star) (650 fork)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
議題類型
refactor
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
under 1 hour
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
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.