merged branch tonypiper/ignore-composer-phar (PR #4034)

Commits
-------

a72d7ef ignore composer.phar
ddca4bb Update CHANGELOG-2.1.md

Discussion
----------

[Global] ignore composer.phar

for running unit tests the external dependencies are managed using composer (since aa055dfd98), so it's necessary to download composer.phar to install them. We don't want composer.phar to be comitted to the codebase so it's necessary to ignore it.

Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -

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

by Seldaek at 2012-04-20T13:19:29Z

I agree it should be ignored, but you also probably should install it somewhere globally on your machine and put that in the PATH so you have it everywhere and don't need 300 copies of it. Incidentally it would also not require you to gitignore it since it would be outside of the git dirs.

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

by tonypiper at 2012-04-20T13:24:56Z

I was going to revise the documentation for running tests and your approach would, of course, be best but I thought a belt-and-braces approach wouldn't hurt...
This commit is contained in:
Fabien Potencier 2012-04-20 16:07:38 +02:00
commit 2c5398e87f
2 changed files with 2 additions and 0 deletions

1
.gitignore vendored
View File

@ -1,4 +1,5 @@
phpunit.xml
composer.lock
composer.phar
autoload.php
/vendor/

View File

@ -203,6 +203,7 @@ To get the diff between two versions, go to https://github.com/symfony/symfony/c
* added a reference to the EventDispatcher on the Event
* added a reference to the Event name on the event
* added fluid interface to the dispatch() method which now returns the Event object
* added GenericEvent event class
### Filesystem