Skip to content

[Doctrine] MakerBundle 1.3 is too old to reference #17022

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

Merged
merged 1 commit into from
Jul 22, 2022

Conversation

jrushlow
Copy link
Contributor

@jrushlow jrushlow commented Jul 20, 2022

MakerBundle 1.2/1.3 were released back in 2018. We're at v1.44.0 now. I don't think we need to reference this anymore.

@carsonbot
Copy link
Collaborator

Hey!

Oh no, it looks like you have made this PR towards a branch that is not maintained anymore. :/
Could you update the PR base branch to target one of these branches instead? 4.4, 5.4, 6.0, 6.1, 6.2.

Cheers!

Carsonbot

@carsonbot carsonbot added this to the 5.0 milestone Jul 20, 2022
@jrushlow jrushlow changed the base branch from 5.0 to 4.4 July 20, 2022 21:22
@jrushlow jrushlow requested a review from xabbuh as a code owner July 20, 2022 21:22
@jrushlow jrushlow changed the base branch from 4.4 to 5.0 July 20, 2022 21:23
@jrushlow jrushlow changed the base branch from 5.0 to 5.4 July 20, 2022 21:23
@jrushlow jrushlow force-pushed the remove-1.3 branch 2 times, most recently from 97e4362 to 7dcac6a Compare July 20, 2022 21:57
- '.. versionadded:: 1.2' # MakerBundle
- '.. versionadded:: 1.11' # MakerBundle
- '.. versionadded:: 1.3' # MakerBundle
- '.. versionadded:: 1.8' # MakerBundle
- '.. versionadded:: 1.11' # Messenger (Middleware / DoctrineBundle)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Version 1.8 is not needed (not referenced in the docs)

@javiereguiluz
Copy link
Member

Jesse, thanks for this contribution. However, it was sent to 5.0 branch, which is no longer maintained. These are the currently maintained branches: https://symfony.com/releases#maintained-symfony-branches Thanks!

Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup

@OskarStark
Copy link
Contributor

@javiereguiluz should we target 4.4 here or do we need 5.4 ?

@OskarStark OskarStark modified the milestones: 5.0, 5.4 Jul 22, 2022
@javiereguiluz
Copy link
Member

Let's merge it in 5.4 😉

Thanks Jesse for yet another nice contribution!

@javiereguiluz javiereguiluz merged commit 16df40b into symfony:5.4 Jul 22, 2022
@jrushlow jrushlow deleted the remove-1.3 branch July 22, 2022 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants