Skip to content

Optimize schedule day iteration#148

Open
sbquinlan wants to merge 1 commit intozslayton:masterfrom
sbquinlan:claude/optimize-schedule-day-iteration-emdvT
Open

Optimize schedule day iteration#148
sbquinlan wants to merge 1 commit intozslayton:masterfrom
sbquinlan:claude/optimize-schedule-day-iteration-emdvT

Conversation

@sbquinlan
Copy link
Copy Markdown
Contributor

This PR does two things:

  1. There's an extra reset_day_of_month() call that can be removed.
  2. Instead of continuing the day_loop in the inner-most loop, we can filter the days of the month in the iterator's implementation.

All tests still pass.

@sbquinlan sbquinlan mentioned this pull request Apr 5, 2026
@sbquinlan sbquinlan force-pushed the claude/optimize-schedule-day-iteration-emdvT branch from 5ff50ae to 7b8c02b Compare April 6, 2026 10:32
@sbquinlan
Copy link
Copy Markdown
Contributor Author

@zslayton gentle ping

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