[PR #10210] Migrate Cache namespace to php 8 #12186

Closed
opened 2026-01-22 16:13:12 +01:00 by admin · 0 comments
Owner

Original Pull Request: https://github.com/doctrine/orm/pull/10210

State: closed
Merged: Yes


This one comes with a controversial change that I did not have the courage to extract to a separate commit PR/commit (feel free to insist I do so): switching @readonly properties to native readonly. I had to tweak some tests to achieve that, and I've extracted that part in a separate commit, which I can backport if you think it should happen.

**Original Pull Request:** https://github.com/doctrine/orm/pull/10210 **State:** closed **Merged:** Yes --- This one comes with a controversial change that I did not have the courage to extract to a separate commit PR/commit (feel free to insist I do so): switching `@readonly` properties to native `readonly`. I had to tweak some tests to achieve that, and I've extracted that part in a separate commit, which I can backport if you think it should happen.
admin added the pull-request label 2026-01-22 16:13:12 +01:00
admin closed this issue 2026-01-22 16:13:12 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#12186