Skip to content

[Messenger] Remove duplicated Redis transport requirements #14984

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

Closed
wants to merge 1 commit into from

Conversation

sebpacz
Copy link
Contributor

@sebpacz sebpacz commented Feb 15, 2021

The Redis transport requirements has been presented twice. I mean the following sentences:

This transport requires the Redis PHP extension (>=4.3) and a running Redis server (^5.0).

To use the Redis transport, you will need the Redis PHP extension (>=4.3) and a running Redis server (^5.0).

The Redis transport requirements has been presented twice. I mean the following sentences: "This transport requires the Redis PHP extension (>=4.3) and a running Redis server (^5.0)." and "To use the Redis transport, you will need the Redis PHP extension (>=4.3) and a running Redis server (^5.0).".
@javiereguiluz
Copy link
Member

Thank you Sebastian.

javiereguiluz added a commit that referenced this pull request Feb 15, 2021
…ts (sebpacz)

This PR was merged into the 5.2 branch.

Discussion
----------

[Messenger] Remove duplicated Redis transport requirements

The Redis transport requirements has been presented twice. I mean the following sentences:

> This transport requires the Redis PHP extension (>=4.3) and a running Redis server (^5.0).

> To use the Redis transport, you will need the Redis PHP extension (>=4.3) and a running Redis server (^5.0).
<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/releases for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `5.x` for features of unreleased versions).

-->

Commits
-------

d6a7314 [Messenger] Remove duplicated Redis transport requirements
@javiereguiluz
Copy link
Member

I'm closing this one manually because GitHub didn't do that automatically after the merge (it was merged here: 23e13fb)

@sebpacz sebpacz deleted the patch-3 branch February 16, 2021 19:41
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.

3 participants