Skip to content
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

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

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

@carsonbot carsonbot commented Jul 20, 2022

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 Jul 20, 2022
@jrushlow jrushlow requested a review from xabbuh as a code owner Jul 20, 2022
@jrushlow jrushlow changed the base branch from 4.4 to 5.0 Jul 20, 2022
@jrushlow jrushlow changed the base branch from 5.0 to 5.4 Jul 20, 2022
@jrushlow jrushlow force-pushed the remove-1.3 branch 2 times, most recently from 97e4362 to 7dcac6a Compare Jul 20, 2022
- '.. 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

@jrushlow jrushlow Jul 20, 2022

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

@javiereguiluz javiereguiluz commented Jul 22, 2022

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

Thanks for the cleanup

@OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Jul 22, 2022

@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

@javiereguiluz javiereguiluz commented Jul 22, 2022

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
3 checks passed
@jrushlow jrushlow deleted the remove-1.3 branch Jul 22, 2022
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.

None yet

4 participants