Skip to content
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

[DIC] Fixed: anonymous services are always private #11666

Closed
wants to merge 2 commits into from

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Aug 13, 2014

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

This PR fix a regression introduce by 33c91f9#diff-c68f5120ea7d664e07e96cc40bdf6295L268

  • First commit is just about CS
  • Second commit really fix the regression (see only this diff to easily understand)

ping @sandermarechal @romainneutron

@romainneutron
Copy link
Contributor

👍

1 similar comment
@nicolas-grekas
Copy link
Member

👍

@nicolas-grekas
Copy link
Member

Thanks @lyrixx.

nicolas-grekas added a commit that referenced this pull request Aug 14, 2014
This PR was squashed before being merged into the 2.5 branch (closes #11666).

Discussion
----------

[DIC] Fixed: anonymous services are always private

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

This PR fix a regression introduce by 33c91f9#diff-c68f5120ea7d664e07e96cc40bdf6295L268

* First commit is just about CS
* Second commit really fix the regression (see only this diff to easily understand)

ping @sandermarechal @romainneutron

Commits
-------

7f4f9ab [DIC] Fixed: anonymous services are always private
@nicolas-grekas
Copy link
Member

Closed via 35994bf

@lyrixx lyrixx deleted the dic-embed-service-private branch August 14, 2014 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants