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

[Console] Single dash for option shortcuts #4346

Merged
merged 2 commits into from May 22, 2012
Merged

[Console] Single dash for option shortcuts #4346

merged 2 commits into from May 22, 2012

Conversation

nanocom
Copy link
Contributor

@nanocom nanocom commented May 20, 2012

See #4062

@travisbot
Copy link

This pull request passes (merged 02290da4 into f433f6b).

@stof
Copy link
Member

stof commented May 20, 2012

please rebase your branch to get rid of these merge commits
btw, you should use feature branches to send your next pull requests instead of using your master branch each time, which limits you to a single PR.

@nanocom
Copy link
Contributor Author

nanocom commented May 20, 2012

Sorry for the mess, cleaning it

@travisbot
Copy link

This pull request passes (merged 63129657 into f433f6b).

@travisbot
Copy link

This pull request passes (merged 2c19b3c into f433f6b).

fabpot added a commit that referenced this pull request May 22, 2012
Commits
-------

2c19b3c Empty shortcut check in the constructor
cf9039e Added a unit test for the shortcut name of the InputOption class

Discussion
----------

[Console] Single dash for option shortcuts

See #4062

---------------------------------------------------------------------------

by travisbot at 2012-05-20T13:09:18Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1380389) (merged 02290da4 into f433f6b).

---------------------------------------------------------------------------

by stof at 2012-05-20T13:16:34Z

please rebase your branch to get rid of these merge commits
btw, you should use feature branches to send your next pull requests instead of using your master branch each time, which limits you to a single PR.

---------------------------------------------------------------------------

by Nanocom at 2012-05-20T13:39:32Z

Sorry for the mess, cleaning it

---------------------------------------------------------------------------

by travisbot at 2012-05-20T13:41:46Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1380549) (merged 63129657 into f433f6b).

---------------------------------------------------------------------------

by travisbot at 2012-05-20T13:43:07Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1380553) (merged 2c19b3c into f433f6b).
@fabpot fabpot merged commit 2c19b3c into symfony:master May 22, 2012
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.

None yet

4 participants