-
Notifications
You must be signed in to change notification settings - Fork 15.6k
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
Fixed the use of c++ keywords within namespace names #15954
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Do I need to do something to fix the issue with failed PHP test before sbenzaquen will approve the merge or should I just wait? |
Thanks for the ping! We temporarily disabled the failing PHP test which was blocking the PR from going through. Rebasing would unstick this, but we've already imported your PR to be merged. No action is needed on your part at this time -- but please give us another ping if you don't see this go through in the next day or so. |
Good afternoon. Sorry for bothering, but is it known, when the commit will be included in the release version? |
This should be picked up in our next quarterly 27.x minor release, which we should start cutting RC's for ~next week! |
Great! Thanks for the info |
…#15954) Closes protocolbuffers#15954 COPYBARA_INTEGRATE_REVIEW=protocolbuffers#15954 from Ilukatsk:main acf3fa4 PiperOrigin-RevId: 611573919
No description provided.