-
Notifications
You must be signed in to change notification settings - Fork 28
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
[1.3.z] Update release checker #1211
[1.3.z] Update release checker #1211
Conversation
@michalvavrik release check failure is expected as we already have similar release |
Won't this PR trigger the release flow which will fail because you create PR from your fork? TBH I'd prefer to actually make release so that next guy that comes after you can just increment by one. This way, we have this whole compatibility issues behind us. |
I create another PR with release after this PR will be merged. So, |
ef6efa2
to
aa70e19
Compare
sounds good |
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
Summary
Backports #1202 and #1209
Please check the relevant options
run tests
phrase in comment)Checklist: