Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[DI][2.7] Include dynamic services in alternatives#19685

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nicolas-grekas merged 1 commit intosymfony:2.7fromro0NL:di/dynamic-service-alternatives
Aug 23, 2016
Merged

[DI][2.7] Include dynamic services in alternatives#19685

nicolas-grekas merged 1 commit intosymfony:2.7fromro0NL:di/dynamic-service-alternatives
Aug 23, 2016

Conversation

@ro0NL
Copy link
Contributor

@ro0NLro0NL commentedAug 20, 2016•
edited
Loading

QA
Branch?2.7
Bug fix?yes
New feature?no
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed ticketscomma-separated list of tickets fixed by the PR, if any
LicenseMIT
Doc PRreference to the documentation PR, if any

sstok, ogizanagi, and linaori reacted with thumbs up emoji
@ogizanagi
Copy link
Contributor

Status: Reviewed

@ro0NL
Copy link
ContributorAuthor

ro0NL commentedAug 20, 2016•
edited
Loading

This will include possible private services as well, but those are still allowed toget.

@ogizanagi
Copy link
Contributor

This will include possible private services as well, but those are still allowed to get.

@ro0NL : And it was already the case anyway, isn't it ?

@ro0NL
Copy link
ContributorAuthor

I guess it depends.. if it was already initialized yes/no. This will include by definition, not by state.

@nicolas-grekas
Copy link
Member

👍

1 similar comment
@stof
Copy link
Member

👍

@nicolas-grekas
Copy link
Member

Thank you@ro0NL.

@nicolas-grekasnicolas-grekas merged commit428b5cc intosymfony:2.7Aug 23, 2016
nicolas-grekas added a commit that referenced this pull requestAug 23, 2016
This PR was merged into the 2.7 branch.Discussion----------[DI][2.7] Include dynamic services in alternatives| Q             | A| ------------- | ---| Branch?       | 2.7| Bug fix?      | yes| New feature?  | no| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets | comma-separated list of tickets fixed by the PR, if any| License       | MIT| Doc PR        | reference to the documentation PR, if anyCommits-------428b5cc include dynamic services in list of alternatives
@ro0NLro0NL deleted the di/dynamic-service-alternatives branchAugust 23, 2016 11:02
This was referencedSep 2, 2016
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ro0NL@ogizanagi@nicolas-grekas@stof@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp