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

Skip to content

[FrameworkBundle] Disable the server:run command when Process component is missing #16870

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 2 commits into from
Dec 7, 2015

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Dec 6, 2015

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

This also backports the improvement for the suggest section from #16650 to the 2.3 branch.

gnugat and others added 2 commits December 6, 2015 23:31
The `server:run` command requires the Process component.
@Tobion
Copy link
Contributor

Tobion commented Dec 6, 2015

👍 Status: Reviewed

@dunglas
Copy link
Member

dunglas commented Dec 7, 2015

👍

@fabpot
Copy link
Member

fabpot commented Dec 7, 2015

Thank you @xabbuh.

@fabpot fabpot merged commit 99d1741 into symfony:2.3 Dec 7, 2015
fabpot added a commit that referenced this pull request Dec 7, 2015
…ess component is missing (gnugat, xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[FrameworkBundle] Disable the server:run command when Process component is missing

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

This also backports the improvement for the `suggest` section from #16650 to the `2.3` branch.

Commits
-------

99d1741 disable server:run cmd without Process component
604174c Suggested Process dependency
@xabbuh xabbuh deleted the pr-16650-2.3 branch December 7, 2015 21:53
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.

6 participants