🥔 Should we keep rebasing 3.0? #6052

Closed
opened 2026-01-22 15:25:36 +01:00 by admin · 5 comments
Owner

Originally created by @guilhermeblanco on GitHub (Aug 28, 2018).

@doctrine/doctrinecore Feel free to vote.

👍 For "Yes, continue"
👎 For "Stop, let's choose another alternative (ie. merge, cherry-pick)"

Originally created by @guilhermeblanco on GitHub (Aug 28, 2018). @doctrine/doctrinecore Feel free to vote. 👍 For "Yes, continue" 👎 For "Stop, let's choose another alternative (ie. merge, cherry-pick)"
admin added the DocumentationQuestion labels 2026-01-22 15:25:36 +01:00
admin closed this issue 2026-01-22 15:25:37 +01:00
Author
Owner

@Ocramius commented on GitHub (Aug 28, 2018):

As discussed in chat, this is currently problematic: we'll most likely need to consider all branches under a certain naming convention (such as (master|\d+.\d+\.x)) as stable.

@Ocramius commented on GitHub (Aug 28, 2018): As discussed in chat, this is currently problematic: we'll most likely need to consider all branches under a certain naming convention (such as `(master|\d+.\d+\.x)`) as stable.
Author
Owner

@lcobucci commented on GitHub (Aug 28, 2018):

We now have 151 PRs targeting master, 30 PRs targeting 2.6, and 8 PRs targeting 2.7 - we have other PRs for older branches but let's just skip them 😜.

I'm pretty sure most of the 151 are already outdated and need to be rebased, but even thought that's the situation I think we should strive to consider master as a stable branch as much as possible.

With the current state of the repo, it's way easier to port a bugfix forward to master than keep it rebasing it.

@lcobucci commented on GitHub (Aug 28, 2018): We now have [151 PRs targeting `master`](https://github.com/doctrine/doctrine2/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+base%3Amaster), [30 PRs targeting `2.6`](https://github.com/doctrine/doctrine2/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+base%3A2.6), and [8 PRs targeting `2.7`](https://github.com/doctrine/doctrine2/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+base%3A2.7) - we have other PRs for older branches but let's just skip them :stuck_out_tongue_winking_eye:. I'm pretty sure most of the 151 are already outdated and need to be rebased, but even thought that's the situation I think we should strive to consider `master` as a stable branch as much as possible. With the current state of the repo, it's way easier to port a bugfix forward to `master` than keep it rebasing it.
Author
Owner

@Majkl578 commented on GitHub (Aug 28, 2018):

On top of what @lcobucci said, I'd also like to mention that 2.6 vs master is not just a few commits and changed files, but a lot of fundamental changes and completely overhauled coding style.
Right now 2.6...master: 1524 files changed, 61877 insertions(+), 78432 deletions(-)

@Majkl578 commented on GitHub (Aug 28, 2018): On top of what @lcobucci said, I'd also like to mention that 2.6 vs master is not just a few commits and changed files, but a lot of fundamental changes and completely overhauled coding style. Right now `2.6...master`: ` 1524 files changed, 61877 insertions(+), 78432 deletions(-)`
Author
Owner

@SenseException commented on GitHub (Sep 6, 2018):

What @Majkl578 says is especially interesting for documentation, that needs to be changed on multiple branches. Currently 2.6 - 3.0, in case changes affects all those versions. We have outdated information in the docs that should get focus like #7374 and #7385.

@SenseException commented on GitHub (Sep 6, 2018): What @Majkl578 says is especially interesting for documentation, that needs to be changed on multiple branches. Currently 2.6 - 3.0, in case changes affects all those versions. We have outdated information in the docs that should get focus like #7374 and #7385.
Author
Owner

@beberlei commented on GitHub (Dec 6, 2020):

Internal question for discussion group.

@beberlei commented on GitHub (Dec 6, 2020): Internal question for discussion group.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#6052