Skip to content

fix: reserve "ReadOnly" keyword for PHP 8.1 and add compatibility#9633

Merged
haberman merged 2 commits intoprotocolbuffers:mainfrom
bshaffer:fix-readonly-php
Apr 12, 2022
Merged

fix: reserve "ReadOnly" keyword for PHP 8.1 and add compatibility#9633
haberman merged 2 commits intoprotocolbuffers:mainfrom
bshaffer:fix-readonly-php

Conversation

@bshaffer
Copy link
Copy Markdown
Contributor

@bshaffer bshaffer commented Mar 15, 2022

see #9629

@bshaffer
Copy link
Copy Markdown
Contributor Author

bshaffer commented Apr 12, 2022

@haberman could we get a review on this one? It's a blocker for a few of our client libraries on PHP 8.1 which contain ReadOnly classes (Firestore, Spanner, and DataStore).

@haberman haberman merged commit eb27c20 into protocolbuffers:main Apr 12, 2022
@bshaffer bshaffer deleted the fix-readonly-php branch April 12, 2022 22:55
haberman added a commit to haberman/protobuf that referenced this pull request May 24, 2022
haberman added a commit that referenced this pull request May 24, 2022
Revert "fix: reserve "ReadOnly" keyword for PHP 8.1 and add compatibi…lity (#9633)"
bshaffer added a commit to bshaffer/protobuf that referenced this pull request May 25, 2022
bshaffer added a commit to bshaffer/protobuf that referenced this pull request May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants