Skip to content

Update composer dependencies, require PHP 8.3+#214

Merged
alies-dev merged 4 commits intomasterfrom
update-dependencies
Feb 10, 2026
Merged

Update composer dependencies, require PHP 8.3+#214
alies-dev merged 4 commits intomasterfrom
update-dependencies

Conversation

@alies-dev
Copy link
Copy Markdown
Collaborator

@alies-dev alies-dev commented Feb 10, 2026

Summary

  • Add @api annotation to public library classes (Link, Google, Ics, Yahoo, InvalidLink) instead of making them final, since they are part of the public API
  • Add typed constants (const string) using PHP 8.3+ syntax to all class constants
  • Add #[\Override] attribute to all methods implementing interfaces or overriding abstract/trait methods
  • Fix InvalidOperand in Ics.php by casting DateInterval::$days to int
  • Make LinkTest final (test class, not public API)
  • Update composer dependencies to require PHP 8.3+ and bump dev dependencies (Psalm 6, PHPUnit 12/13)
  • Update CI matrix to test PHP 8.3, 8.4, 8.5

Test plan

  • Run Psalm and verify all 34 errors are resolved
  • Run PHPUnit test suite to ensure no regressions

🤖 Generated with Claude Code

alies-dev and others added 4 commits February 10, 2026 09:59
PHPUnit 12 dropped support for @test doc annotations. Use the
#[Test] attribute from PHPUnit\Framework\Attributes instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alies-dev alies-dev changed the title Fix Psalm errors: add @api, typed constants, #[Override] Update composer dependencies, require PHP 8.3+ Feb 10, 2026
@alies-dev alies-dev self-assigned this Feb 10, 2026
@alies-dev alies-dev merged commit 612dcfa into master Feb 10, 2026
14 of 15 checks passed
@alies-dev alies-dev deleted the update-dependencies branch February 10, 2026 09:16
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