Skip to content

Comments

docs: add migration guide for v4 to v5#134

Merged
gsteel merged 4 commits intolaminas:5.0.xfrom
mairo744:docs/migration-guide-v5
Dec 3, 2025
Merged

docs: add migration guide for v4 to v5#134
gsteel merged 4 commits intolaminas:5.0.xfrom
mairo744:docs/migration-guide-v5

Conversation

@mairo744
Copy link

@mairo744 mairo744 commented Jul 8, 2025

Q A
Documentation yes
Bugfix no
BC Break no
New Feature no
RFC no
QA no

Description

This pull request adds a migration guide for users upgrading from version 4 to version 5 of the laminas-hydrator package.

The new guide includes:

  • Clear explanation of removed and replaced hydrator classes.
  • Notes on removed legacy Zend aliases and how to update plugin manager usage.
  • Clarification on removed exceptions and updated SerializableStrategy behavior.
  • Recommendations to avoid inheritance and prefer composition or decorators going forward.

Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mairo744! LGTM. Paging @froschdesign for review too.

@gsteel gsteel requested a review from froschdesign July 9, 2025 08:59
mmalac added 2 commits July 9, 2025 12:44
Signed-off-by: mmalac <mmalac@sygic.com>
Signed-off-by: mmalac <mmalac@sygic.com>
@mairo744 mairo744 force-pushed the docs/migration-guide-v5 branch from e7cd88c to 9202f2a Compare July 9, 2025 10:44
@mairo744 mairo744 requested a review from froschdesign July 10, 2025 19:35
@mairo744
Copy link
Author

Is there anything I can improve or help with for v5? I'm happy to assist however I can. 😊

@gsteel
Copy link
Member

gsteel commented Jul 15, 2025

Is there anything I can improve or help with for v5? I'm happy to assist however I can. 😊

Thanks @mairo744 - I'm not overly familiar with this lib, but we will definitely want to at least:

  • Update the entire codebase to use native parameter and return types
  • Make any improvements to QA tools such as adding composer-require-checker (Example: Setup standalone installation of composer-require-checker laminas-view#274)
  • Get dev dependencies upgraded to whatever is supported between PHP 8.1 - 8.4, i.e. Psalm, PHPUnit etc
  • Audit require and refactor to avoid soft dependencies, or minimise dependencies where possible
  • Fix outstanding issues/bugs
  • Reduce the Psalm baseline to as close to zero as possible

@mairo744
Copy link
Author

Thank you for the detailed tips — really appreciate it.
I’ll do my best to help with these and I open PRs incrementally as I go. 😊

Copy link
Member

@froschdesign froschdesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only small changes are required.

…Support

Signed-off-by: mmalac <mmalac@sygic.com>
@mairo744 mairo744 force-pushed the docs/migration-guide-v5 branch from a9f59e9 to ce72406 Compare August 18, 2025 19:02
Copy link
Member

@froschdesign froschdesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏻

@gsteel gsteel self-assigned this Dec 3, 2025
@gsteel gsteel merged commit d7ea4a4 into laminas:5.0.x Dec 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants