Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove UPGRADE-5.4.md #44489

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 8, 2021
Merged

Remove UPGRADE-5.4.md #44489

merged 1 commit into from
Dec 8, 2021

Conversation

fancyweb
Copy link
Contributor

@fancyweb fancyweb commented Dec 7, 2021

Q A
Branch? 6.0
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

@carsonbot carsonbot added this to the 6.0 milestone Dec 7, 2021
UPGRADE-6.0.md Outdated
@@ -173,6 +173,7 @@ Messenger
* Removed the `prefetch_count` parameter in the AMQP bridge.
* Removed the use of TLS option for Redis Bridge, use `rediss://127.0.0.1` instead of `redis://127.0.0.1?tls=1`
* The `delete_after_ack` config option of the Redis transport now defaults to `true`
* The `reset_on_message` config option now defaults to `true`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This line is in branch 5.4 but not in 6.0, both files should logically be the same.

Copy link
Contributor Author

@fancyweb fancyweb Dec 7, 2021

Choose a reason for hiding this comment

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

Oh probably because it's a FWB config and not related to the Messenger component? So it should be removed in 5.4 instead?

Copy link
Member

Choose a reason for hiding this comment

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

see #43203

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It needs to be removed on 5.4 then.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done in #44496

@fancyweb fancyweb force-pushed the minor/fix-upgrade-6- branch from e1c7982 to ff68d01 Compare December 7, 2021 12:54
@fancyweb fancyweb force-pushed the minor/fix-upgrade-6- branch from ff68d01 to 4af20e4 Compare December 7, 2021 12:54
@fancyweb fancyweb changed the title Remove UPGRADE-5.4.md and add missing entry in UPGRADE-6.0.md Remove UPGRADE-5.4.md Dec 7, 2021
@ogizanagi
Copy link
Contributor

Thanks @fancyweb.

@ogizanagi ogizanagi merged commit 4b16533 into symfony:6.0 Dec 8, 2021
@fancyweb fancyweb deleted the minor/fix-upgrade-6- branch December 8, 2021 13:56
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