propelorm / Propel2

Propel2 is an open-source high-performance Object-Relational Mapping (ORM) for modern PHP
http://propelorm.org/
MIT License
1.26k stars 399 forks source link

Fix up PHPStan errors. #1955

Closed michbeck closed 1 year ago

michbeck commented 1 year ago

Road to level 8

dereuromark commented 1 year ago

The other changes look good to me.

codecov-commenter commented 1 year ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 79.48718% with 8 lines in your changes missing coverage. Please review.

Project coverage is 89.27%. Comparing base (011aaa7) to head (e6f2a49). Report is 61 commits behind head on master.

Files with missing lines Patch % Lines
...nable/VersionableBehaviorObjectBuilderModifier.php 0.00% 6 Missing :warning:
src/Propel/Generator/Manager/ModelManager.php 75.00% 1 Missing :warning:
...e/ActiveQuery/Criterion/AbstractModelCriterion.php 0.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1955 +/- ## ============================================ + Coverage 88.56% 89.27% +0.71% + Complexity 8051 8049 -2 ============================================ Files 243 232 -11 Lines 24590 24519 -71 ============================================ + Hits 21777 21889 +112 + Misses 2813 2630 -183 ``` | [Flag](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | Coverage Δ | | |---|---|---| | [5-max](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `89.27% <79.48%> (+0.71%)` | :arrow_up: | | [7.4](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `89.27% <79.48%> (+0.71%)` | :arrow_up: | | [agnostic](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `67.47% <58.97%> (+0.17%)` | :arrow_up: | | [mysql](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `69.81% <51.28%> (+0.72%)` | :arrow_up: | | [pgsql](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `69.83% <56.41%> (+0.72%)` | :arrow_up: | | [sqlite](https://app.codecov.io/gh/propelorm/Propel2/pull/1955/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm) | `67.80% <51.28%> (+0.72%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=propelorm#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

michbeck commented 1 year ago

@mringler @dereuromark can we move forward here?

mringler commented 1 year ago

@michbeck It looks like @dereuromark has left the project, and it looks like there is no successor at Spryker at the moment. I have contacted the owner, and he told me that the project will keep being maintained by Spryker, but I haven't heard back if/when there will be someone who actively maintains Propel2, i.e. merges this PRs. For what it's worth, I am very sorry for the situation! I really appreciate the work you put into this in the previous PRs, and I would have really liked to see this merged, preferably in acceptable time. I guess when/if someone takes over, contributors like you and I can decide if they want to come back. Best wishes

floriankraemer commented 1 year ago

@mringler thank you for the reminder and your patience! ❤️ I really hope you'll keep contributing after this, I'm really sorry that you had to wait, I know from personal experience that it is frustrating when your contributions won't move forward. 😞

I'll take this over temporarely until I figure out who will be our new internal owner. Your PR looks good to me! 💪 There is just one comment from Mark, to which I agree, that needs to be fixed but we could do that as well.

@michbeck can you maybe take care of it tomorrow? I'll approve the merge as soon as this has been fixed by someone and I've got the required permissions for this repository.

floriankraemer commented 1 year ago

@gechetspr this is ready for merge now.