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

Removed code references to Symfony Standard Distribution #3427

Conversation

dcsg
Copy link
Member

@dcsg dcsg commented Jan 5, 2014

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets

After talk with @wouterj on #3426 and IRC about references in the standalone Console component to the framework I decided to submit this PR to change the code that have some references to the Symfony Standard Distribution in the code.

@@ -84,11 +84,10 @@ and add the following to it::
You also need to create the file to run at the command line which creates
an ``Application`` and adds commands to it::

#!/usr/bin/env php
<?php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should remove this tag too, it breaks highlighting.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

@weaverryan
Copy link
Member

Awesome 👍! I just had one quick question I posted :)

@dcsg
Copy link
Member Author

dcsg commented Jan 9, 2014

@weaverryan @wouterj reverted the shebang! Let me know if it's ok now.

Also I rebased and squashed it!

@xabbuh
Copy link
Member

xabbuh commented Jan 9, 2014

👍 looks good now!

@weaverryan
Copy link
Member

Yes, it's great now - thanks for your hard work - it's a nice detail to have right :). Cheers!

weaverryan added a commit that referenced this pull request Jan 9, 2014
…anielcsgomes)

This PR was merged into the 2.3 branch.

Discussion
----------

Removed code references to Symfony Standard Distribution

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets |

After talk with @wouterj on #3426 and IRC about references in the standalone Console component to the framework I decided to submit this PR to change the code that have some references to the Symfony Standard Distribution in the code.

Commits
-------

4fd7022 Removed code references to Symfony Standard Distribution
@weaverryan weaverryan merged commit 4fd7022 into symfony:2.3 Jan 9, 2014
@dcsg dcsg deleted the remove-component-references-to-symfony-framework branch January 9, 2014 18:59
@dcsg
Copy link
Member Author

dcsg commented Jan 9, 2014

👍

@dcsg dcsg restored the remove-component-references-to-symfony-framework branch January 12, 2014 20:41
@dcsg dcsg deleted the remove-component-references-to-symfony-framework branch January 12, 2014 20:43
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.

6 participants