tito10047 / php-calendar / 28450503281

84%

main: 84%

Build Type

Pull #3

github

Committed by tito10047

Commit Message

refactor: rename count() to limitTo() and add input validation

count() risks a signature clash if RecurrenceRule ever implements \Countable
(\Countable::count() must return int with no parameters).  limitTo(int)
is also more expressive at the call-site.

Guards against limitTo(0) / limitTo(-1): zero or negative occurrences is
almost certainly a caller bug, so we throw \InvalidArgumentException early
rather than silently producing an empty expansion.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #3: feat!: v2.0 — pure-library core, iCal, recurrence, resource calendar

Coverage Stats

629 of 747 new or added lines in 12 files covered. (84.2%)

7 existing lines in 1 file now uncovered.

681 of 809 relevant lines covered (84.18%)

7.31 hits per line