Skip to content

Support readonly#659

Open
Nek- wants to merge 6 commits intophpspec:masterfrom
Nek-:support-readonly
Open

Support readonly#659
Nek- wants to merge 6 commits intophpspec:masterfrom
Nek-:support-readonly

Conversation

@Nek-
Copy link
Copy Markdown
Contributor

@Nek- Nek- commented Nov 14, 2025

This PR is supposed to fix #586 and replace #623 .

@Nek- Nek- marked this pull request as ready for review November 21, 2025 18:38
@Nek-
Copy link
Copy Markdown
Contributor Author

Nek- commented Nov 21, 2025

@stof this PR is ready for review!

Comment thread src/Prophecy/Doubler/Generator/Node/PropertyNode.php Outdated
Comment thread src/Prophecy/Doubler/Generator/Node/PropertyNode.php Outdated
Comment thread src/Prophecy/Doubler/Generator/Node/PropertyNode.php Outdated
Comment thread src/Prophecy/Doubler/ObjectProphecyClosureContainer.php Outdated
Nek- added a commit to Nek-/prophecy that referenced this pull request Mar 21, 2026
Nek- added a commit to Nek-/prophecy that referenced this pull request Mar 21, 2026
@Nek- Nek- force-pushed the support-readonly branch from b024c9b to 4c17697 Compare March 21, 2026 19:16
@Nek-
Copy link
Copy Markdown
Contributor Author

Nek- commented Mar 21, 2026

feedback done @stof

@Nek- Nek- force-pushed the support-readonly branch from 4c17697 to 8fcd94f Compare March 21, 2026 19:18
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.

Mocking PHP 8.2 readonly classes generates readonly double with untyped property and fails

3 participants