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
DX: replace fixture classes with anonymous ones #7533
DX: replace fixture classes with anonymous ones #7533
Conversation
can you gather all of those in one PR? |
private function createStdinStreamDouble(): object | ||
{ | ||
return new class() { |
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.
I know it was not there initially, but can we somehow add any typehint here other than object
?
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.
Unfortunately no, https://github.com/composer/composer/blob/2.6.6/src/Composer/Installer/BinaryInstaller.php#L248 does not have it either.
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.
overall, 👍🏻
I left few small comments and disabled automerge, feel free to decide to incorporate them or not and merge afterwards
Co-authored-by: Dariusz Rumiński <dariusz.ruminski@gmail.com>
No description provided.