-
Notifications
You must be signed in to change notification settings - Fork 321
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
prepare main for 1.4 dev #3092
prepare main for 1.4 dev #3092
Conversation
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.
@curtbushko this look okay for the cloud tests? |
@wilkermichael Yes, that looks right. I guess we will have to wait until main on every other project is updated with the correct versions too. |
I made some changes to the PR to be consistent with having the minor version of the tag. I also remove the last usages of |
I will also cherry-pick the docker changes to other branches. |
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.
As long as versions have -dev
, I think we are good.
157be8c
to
cb3597a
Compare
7a12c17
to
2e58f6c
Compare
Changes proposed in this PR:
How I've tested this PR:
How I expect reviewers to test this PR:
Checklist: