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.