Skip to content

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

Merged
taylorotwell merged 1 commit into
laravel:12.xfrom
kayw-geek:feature/fix-incorrect-carbon-phpdoc
Mar 24, 2025
Merged

[12.x] Update DateFactory method annotations for Carbon v3 compatibility#55151
taylorotwell merged 1 commit into
laravel:12.xfrom
kayw-geek:feature/fix-incorrect-carbon-phpdoc

Conversation

@kayw-geek

Copy link
Copy Markdown
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
Copy Markdown
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
calebdw added a commit to calebdw/larastan that referenced this pull request Apr 1, 2025
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
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
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.

2 participants