Skip to content

Updated Factory Interface Methods#615

Merged
ramsey merged 2 commits intoramsey:5.xfrom
MGatner:5.x
Dec 14, 2025
Merged

Updated Factory Interface Methods#615
ramsey merged 2 commits intoramsey:5.xfrom
MGatner:5.x

Conversation

@MGatner
Copy link
Copy Markdown

@MGatner MGatner commented Aug 10, 2025

  • Adds the existing factory methods for uuid7() and uuid8() to the interface.

Description

UuidFactoryInterface was intentionally left unchanged in v4 releases when v7 & v8 methods were added to avoid a BC. With version 5 of this library the interface should be updated (or a new interface added).

Motivation and context

The interface(s) will now correctly reflect the available methods from the factory.

How has this been tested?

  • Tested in-place with an existing v4 installation
  • Verified fixes the PHPStan complaint for method.notFound

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.

@MGatner MGatner requested a review from ramsey as a code owner August 10, 2025 12:30
@ramsey
Copy link
Copy Markdown
Owner

ramsey commented Aug 17, 2025

The static analysis build is failing, and the errors don't appear (at a glance) related to your changes, so I'll need to figure out what's going on before I merge this.

@ramsey ramsey merged commit 4109ad8 into ramsey:5.x Dec 14, 2025
5 of 6 checks passed
@ramsey
Copy link
Copy Markdown
Owner

ramsey commented Dec 14, 2025

I fixed the issues in the 5.x branch and merged this in commit 44318fd.

Thank you for contributing! 🎉

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