Skip to content
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

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

Merged
merged 2 commits into from
Apr 12, 2022

Conversation

bshaffer
Copy link
Contributor

@bshaffer bshaffer commented Mar 15, 2022

see #9629

@bshaffer
Copy link
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