Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Cron explainer says "on weekdays" for `0 0 1-31 * 1-5`, but the next runs include weekends

オープン
#329 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@Saifjafri212213 がすでに取り組んでいます。

2026年10月23日 から。

評価

この issue はまだ評価されていません。

説明

bug hacktoberfest help wanted

Enter 0 0 1-31 * 1-5 in the cron explainer. The description says "At 00:00 on weekdays", but the next runs listed from Friday, Oct 23 are Sat 24, Sun 25, Mon 26, Tue 27 and Wed 28. The run list is right: when the day-of-month and day-of-week fields are both set (neither starts with *), cron runs on days that match either one, and 1-31 matches every day. The description is what's off. describeDays (lib/cron.ts, lines 265 to 291) treats a field that covers every value as unrestricted, while nextRuns (line 366) checks whether the field starts with * (star, line 134), the way cron does.

What to do

  • Decide "restricted" in describeDays the same way nextRuns does, and describe what actually runs. For 0 0 1-31 * 1-5 that's every day.
  • 0 0 * * 1-5 should still say "on weekdays".

Where to look

  • lib/cron.ts: describeDays (lines 265 to 291), describeCron (line 294) and nextRuns (line 366)
  • lib/__tests__/cron.test.ts

Acceptance criteria

  • The description of 0 0 1-31 * 1-5 matches its next runs
  • 0 0 * * 1-5, 0 0 1 * 1 and the other existing descriptions stay the same
  • Tests that fail on the current code
  • npm run lint, npm run test and npm run build pass

Want to work on this? Comment below and I'll assign it to you. Please take one issue at a time, branch off develop, and open your PR against develop with Closes #329 in the description.

主要言語
TypeScript
スター
1
フォーク
9
平均マージ
7時間 30分
マージ済み PR(30日)
151

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

codercops/toolbelt のほかの issue

codercops/toolbelt の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。