-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Fix versions in Upgrading Major Version #13029
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
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.
Thank you for that PR!
It seems that a block below is duplicated, could you please remove the lines 155 to 167, then add the diff on line 177 too?
You can also target 4.4 for this fix, or we can do it for you on merge.
@HeahDude Done! I can target 4.4, but then you'd have to merge it to master as well, am I mistaken? |
We merge this afterwards to master |
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.
Thank you!
So should I retarget to |
You can if you want, but we use an internal tool to merge PRs and can do it otherwise, thanks. |
Thank you Benjamin. |
This PR was submitted for the master branch but it was squashed and merged into the 4.4 branch instead (closes #13029). Discussion ---------- Fix versions in Upgrading Major Version This PR fixes the versions of the upgrade: `4.4 -> 5.0` instead of `4.3 -> 4.4`. I'm also suggesting a change to the following wording: > (...) follow their own versioning scheme. to make the sentence clearer. Commits ------- 19e1ec7 Fix versions in Upgrading Major Version
This PR fixes the versions of the upgrade:
4.4 -> 5.0
instead of4.3 -> 4.4
.I'm also suggesting a change to the following wording:
to make the sentence clearer.