Skip to content

Remove reference to 2.x, 3.x, 4.x versions #17589

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
Dec 22, 2022

Conversation

alamirault
Copy link
Contributor

@alamirault alamirault commented Dec 18, 2022

closes #17583

@carsonbot carsonbot added this to the 6.3 milestone Dec 18, 2022
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.

@xabbuh not sure about the removal of the 2 files

@xabbuh
Copy link
Member

xabbuh commented Dec 21, 2022

looks good to me, I don't think we need redirects for the two removed chapters

@javiereguiluz @wouterj What do you think?

@javiereguiluz javiereguiluz modified the milestones: 6.3, 6.0 Dec 22, 2022
@javiereguiluz javiereguiluz changed the base branch from 6.3 to 6.0 December 22, 2022 19:02
@javiereguiluz
Copy link
Member

I agree with @xabbuh: we "soft removed" those two articles in 4.0 version, so we can now fully remove them.

I'm merging this in 6.0 branch and up. Thanks Antoine for an excellent contribution!

@javiereguiluz javiereguiluz force-pushed the feature/remove-eol-references branch from 8b65c0c to 98a1dca Compare December 22, 2022 19:02
@javiereguiluz javiereguiluz merged commit 5083cad into symfony:6.0 Dec 22, 2022
@alamirault alamirault deleted the feature/remove-eol-references branch December 22, 2022 20:32
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.

5 participants