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

Appearance settings

Fixed a typo in "define commands as service" #7124

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
Nov 10, 2016
Merged

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Nov 7, 2016

In the introduction of this chapter, we are explaining the restrictions of the "normal" way to register commands, before showing that defining them as services can solve these problems, so at this point we don't talk about a service but a basic command.

@ogizanagi
Copy link
Contributor

👍

Status: Reviewed

@javiereguiluz
Copy link
Member

👍 it makes sense!

Thanks Robin.

@wouterj
Copy link
Member

wouterj commented Nov 10, 2016

Thanks Robin!

@wouterj wouterj merged commit 43ecb46 into symfony:2.7 Nov 10, 2016
wouterj added a commit that referenced this pull request Nov 10, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

Fixed a typo in "define commands as service"

In the introduction of this chapter, we are explaining the restrictions of the "normal" way to register commands, before showing that defining them as services can solve these problems, so at this point we don't talk about a service but a basic command.

Commits
-------

43ecb46 Fixed a typo in "define commands as service"
@chalasr chalasr deleted the patch-1 branch November 10, 2016 20:33
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.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.