As described in#28275, all of the configuration can be done using yield and that we could remove the support for other ways (especially the obscure return[['method', -10]] syntax) as I believe this would clarify the configuration a lot.

Remove the "obscure" message subscriber configuration

cf2ad86

carsonbot added Status: Needs Review Messenger BC Break Status: Reviewed and removed Status: Needs Review labels

Aug 28, 2018

fabpot approved these changes

Aug 29, 2018

View reviewed changes

Copy link

Member

fabpot commentedAug 29, 2018

Thank you@sroze.

fabpot merged commitcf2ad86 intosymfony:master

Aug 29, 2018

fabpot added a commit that referenced this pull request

Aug 29, 2018

feature#28294[Messenger] Remove the "obscure" message subscriber co…

8651758

…nfiguration (sroze)This PR was merged into the 4.2-dev branch.Discussion----------[Messenger] Remove the "obscure" message subscriber configuration| Q             | A| ------------- | ---| Branch?       | master| Bug fix?      | no| New feature?  | no| BC breaks?    | yes| Deprecations? | no| Tests pass?   | yes| Fixed tickets | ø| License       | MIT| Doc PR        | øAs described in#28275, all of the configuration can be done using yield and that we could remove the support for other ways (especially the obscure return `[['method', -10]]` syntax) as I believe this would clarify the configuration a lot.Commits-------cf2ad86 Remove the "obscure" message subscriber configuration

sroze deleted the remove-obscure-subscriber-configuration-method branch

August 29, 2018 15:28

javiereguiluz mentioned this pull request

Sep 7, 2018

[Messenger] Remove the "obscure" message subscriber configurationsymfony/symfony-docs#10278

Closed

Copy link

Member

javiereguiluz commentedSep 7, 2018

I've createdsymfony/symfony-docs#10278 to document this new feature. Please, don't forget to create a doc issue for every new feature.

The Messenger component docs are not great yet, so I don't know if this was documented and we need to change it or it was never documented and need to add it. Thanks!