Skip to content

Fix hydration dependency#35

Open
datasage wants to merge 10 commits into
laminas-api-tools:2.5.xfrom
datasage:fix-hydration-depdency
Open

Fix hydration dependency#35
datasage wants to merge 10 commits into
laminas-api-tools:2.5.xfrom
datasage:fix-hydration-depdency

Conversation

@datasage

@datasage datasage commented Jan 2, 2023

Copy link
Copy Markdown
Contributor
Q A
Documentation no
Bugfix yes
BC Break yes
New Feature yes
RFC no
QA no

Description

The work in this branch targets the abandoned dependency of phpro/zf-doctrine-hydration-module along with any other changes necessary to allow an updated version of the dependency to work with this library. I've taken the abandoned dependency and forked it.

When possible a range of versions were used in dependencies. The most significant changes are the following:

  • PHP version minimum is now 7.4
  • Doctrine ODM now requires 2.0.
  • Several modules have had minimum versions increased due to refactoring or adding typehints

There is some more work to be done. Such as fixing incomplete tests, fixing psalm errors, etc. My goal was to keep the changes limited and address some of those other issues in additional PR's.

@javabudd

Copy link
Copy Markdown

@datasage looks like some of your commits need to be signed off: https://github.com/laminas-api-tools/api-tools-doctrine/pull/35/checks?check_run_id=10399596166

@javabudd

Copy link
Copy Markdown

This PR and #29 look like they have the same goals in mind. @Ocramius We may need direction here as to which PR should be focused on.

@datasage

Copy link
Copy Markdown
Contributor Author

@datasage looks like some of your commits need to be signed off: https://github.com/laminas-api-tools/api-tools-doctrine/pull/35/checks?check_run_id=10399596166

I will look at getting this handed

This PR and #29 look like they have the same goals in mind. @Ocramius We may need direction here as to which PR should be focused on.

My goal was to address the immediate dependency issues with minimal code changes needed to address any compatibility issues. There are certainly further improvements that could be made, such as fully refactoring away from phpro/zf-doctrine-hydration-module (I switched to a forked one for now with minimal updates to avoid any breaks).

…ew version that is now current.

- Due to transitive dependencies PHP 7.4 will be new minimum.

Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
…sue with test combination.

Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
Signed-off-by: Dan Johansson <djohansson@tnsinc.com>
@datasage datasage force-pushed the fix-hydration-depdency branch from 4e50034 to f6e8bde Compare September 15, 2023 18:29
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