-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
PHP: attempt to add v1.21.4 to interop matrix #19228
PHP: attempt to add v1.21.4 to interop matrix #19228
Conversation
#19230 would need to be backported to make this PR work. |
424922b
to
e7cb1f0
Compare
e7cb1f0
to
cd591f0
Compare
@jtattermusch This is ready for another review. The Adhoc interop matrix job was succesful - I manually triggered another build, PTAL. Thanks. |
@@ -281,8 +281,7 @@ def __init__(self, patch=[], runtimes=[], testcases_file=None): | |||
('v1.16.0', ReleaseInfo(testcases_file='php__v1.0.1')), | |||
('v1.17.1', ReleaseInfo(testcases_file='php__v1.0.1')), | |||
('v1.18.0', ReleaseInfo()), | |||
# TODO:https://github.com/grpc/grpc/issues/18264 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave a comment why v1.19 an v1.20 are missing so it's obvious this is not a mistake.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a comment.
cd591f0
to
e92622e
Compare
No description provided.