New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable check for security vulnerabilites step #15527
Conversation
Wojdylak
commented
Nov 14, 2023
Q | A |
---|---|
Branch? | 1.12 |
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | N/A |
License | MIT |
Bunnyshell Preview Environment deletedAvailable commands:
|
1226356
to
5a1f03c
Compare
Thank you, Karol! 🎉 |
|
||
- `doctrine/doctrine-bundle:2.11.0`: | ||
|
||
This version uses the [readonly property](https://github.com/doctrine/DoctrineBundle/blob/2.11.0/Repository/ServiceEntityRepositoryProxy.php#L34), which is available from PHP 8.1 while the package also supports PHP 7.4 and 8.0, leading to compatibility issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This really shouldn't be needed, the file you're linking is conditionally loaded by the "real" class downstream users are meant to be using, and this case it would be only when doctrine/orm:^3.0
is installed (which requires PHP 8.1 anyway). Sounds like this might be another case of a static analysis tool being over-eager?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We managed to solve it in #15531. Thanks for rising this topic :)!
…dylak) This PR was merged into the 1.12 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.12 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | #15527 | License | MIT [link](#15527 (comment)) Commits ------- Add ignore parse errors to composer require checker