Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12.x] Update DateFactory method annotations for Carbon v3 compatibility #55151

Merged

Conversation

kayw-geek
Copy link
Contributor

Description

Synchronizes method annotations in DateFactory with the latest Carbon v3 Factory class. This ensures our type definitions exactly match the upstream Carbon library's implementation.

Reference: briannesbitt/Carbon Factory.php

Impact

This is a documentation-only change that ensures our type definitions stay in sync with Carbon. It improves:

  • IDE autocompletion accuracy
  • Static analysis reliability
  • Type checking precision

No runtime behavior is affected.

@kayw-geek kayw-geek changed the title Update DateFactory method annotations for Carbon v3 compatibility [12.x] Update DateFactory method annotations for Carbon v3 compatibility Mar 24, 2025
@kayw-geek
Copy link
Contributor Author

Should we consider using the @mixin \Carbon\Factory to replace these @method?

@taylorotwell taylorotwell merged commit 964c4bb into laravel:12.x Mar 24, 2025
41 checks passed
calebdw added a commit to calebdw/larastan that referenced this pull request Apr 1, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The methods `maxValue` and `minValue` are not available in the `Date`
class. This removes the assertions for these methods.

See: laravel/framework#55151
calebdw added a commit to calebdw/larastan that referenced this pull request Apr 1, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The methods `maxValue` and `minValue` are not available in the `Date`
class. This removes the assertions for these methods.

See: laravel/framework#55151
nunomaduro pushed a commit to larastan/larastan that referenced this pull request Apr 3, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The methods `maxValue` and `minValue` are not available in the `Date`
class. This removes the assertions for these methods.

See: laravel/framework#55151
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.

None yet

2 participants