[Messenger] Rename Adapters to Transports #27129
Conversation
3220182 to
68208f3
Compare
|
What I also found is that namespaces of interfaces and implementations are not consistent. |
|
So I'd propose
|
|
I agree with @Tobion . But I'd keep |
7d8a7f1 to
2c7730e
Compare
|
Good catch @Tobion, it was the right time to move them. I've moved the handler and middleware ones, kept the Bus one at the root like we do on other bundles like the Serializer. |
2c7730e to
13b7475
Compare
|
Thank you @sroze. |
This PR was squashed before being merged into the 4.1-dev branch (closes #27129). Discussion ---------- [Messenger] Rename Adapters to Transports | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | ø | License | MIT | Doc PR | ø Last of our tasks on the "plan to beta", renaming "adapters" to "transports". This is a term that makes more sense and is commonly used within the "queue community". Commits ------- 13b7475 [Messenger] Rename Adapters to Transports
| @@ -60,16 +60,16 @@ | |||
| </service> | |||
|
|
|||
| <!-- Adapters --> | |||
There was a problem hiding this comment.
Does this line also should be renamed?
This PR was submitted for the master branch but it was squashed and merged into the 4.1 branch instead (closes #9727). Discussion ---------- [Messenger] Update the messenger documentation - [x] Fixes #9641 with the middleware configuration. - [x] Fixes #9617 with the multiple bus configuration. - [x] Change adapters to transports (waiting merge: symfony/symfony#27129) - [x] middlewares config entry is renamed middleware (symfony/symfony#27177) - [x] in the config, message buses names are the full service id you'll use (symfony/symfony#27162) - [x] Add TransportInterface as first class citizen sender+receiver (symfony/symfony#27164) Commits ------- c3c3528 Few updates following review 64bfd75 Change wording and don't use `.`-based services so it's just clearer e1f3b5a Fix the formating of the method name 9b7b85f Update the example of using multiple buses to use DI's `bind`s c76b2c2 Uses the full service name when configuring the buses from the YAML configuration 2409798 Middleware does not have a plural a20286d Add a note about the symfony serializer pack 10f46eb Introduce the `TransportInterface` ef70bc0 Add a documentation about the middlewares 3ff8cfe Add multiple buses configuration and type-hint example a4bc592 Rename the adapters to transport
Last of our tasks on the "plan to beta", renaming "adapters" to "transports". This is a term that makes more sense and is commonly used within the "queue community".