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

Skip to content

[Contracts] Add traits+interfaces from the DI component #28206

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
Sep 4, 2018

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Aug 16, 2018

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -
  • added Service\ServiceSubscriberInterface to declare the dependencies of a class that consumes a service locator
  • added Service\ServiceSubscriberTrait to implement Service\ServiceSubscriberInterface using methods' return types
  • added Service\ServiceLocatorTrait to help implement PSR-11 service locators

@nicolas-grekas nicolas-grekas added this to the next milestone Aug 16, 2018
@nicolas-grekas nicolas-grekas force-pushed the contract-di branch 10 times, most recently from 2701107 to 1cdc729 Compare August 19, 2018 13:00
@nicolas-grekas nicolas-grekas force-pushed the contract-di branch 5 times, most recently from 7fdf5c1 to d4256fe Compare September 4, 2018 08:16
@nicolas-grekas
Copy link
Member Author

(rebased)

@fabpot
Copy link
Member

fabpot commented Sep 4, 2018

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 675abdc into symfony:master Sep 4, 2018
fabpot added a commit that referenced this pull request Sep 4, 2018
…t (nicolas-grekas)

This PR was merged into the 4.2-dev branch.

Discussion
----------

[Contracts] Add traits+interfaces from the DI component

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

  * added `Service\ServiceSubscriberInterface` to declare the dependencies of a class that consumes a service locator
 * added `Service\ServiceSubscriberTrait` to implement `Service\ServiceSubscriberInterface` using methods' return types
 * added `Service\ServiceLocatorTrait` to help implement PSR-11 service locators

Commits
-------

675abdc [Contracts] Add traits+interfaces from the DI component
fabpot added a commit that referenced this pull request Sep 5, 2018
This PR was merged into the 4.2-dev branch.

Discussion
----------

[DI] leverage Contracts\Service

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Embedding #28206 for now.
This deprecates some generic interfaces from the DI component in favor of the similar ones proposed for `Contracts\Service`.

Commits
-------

87392ab [DI] leverage Contracts\Service
@nicolas-grekas nicolas-grekas deleted the contract-di branch September 5, 2018 09:07
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.2 Nov 1, 2018
This was referenced Nov 3, 2018
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