Add .php_cs file to test CS of this repository
Add visibility fixer for method/property visibilities
Boost priority of the indentation fixer

Many other fixers have the assumption that this has been ran, so it should run early
Avoid replacing nothing
Boost priority of other whitespace fixers
merged branch Seldaek/visibility (PR #36)

Commits
-------

b23f292 Boost priority of other whitespace fixers
d950a6e Avoid replacing nothing
ca93e4b Boost priority of the indentation fixer
7eafee8 Add visibility fixer for method/property visibilities

Discussion
----------

VisibilityFixer

PSR-2 compliant

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

by jalliot at 2012-05-22T15:21:22Z

Looking at PSR-2 (and your code), visibility must be set even on methods on interfaces.
AFAIK this goes against Symfony2 CS which never sets visibility on interfaces (since all methods are public).
Is it really the intended behaviour?

/cc @fabpot @stof

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

by Seldaek at 2012-05-22T15:23:31Z

Well, PSR-2 states that, so I'd say we just fix it and update Symfony CS and be done with it.
updated the phar
merged branch jonathaningram/php_cs_file (PR #35)

Commits
-------

9119079 Add .php_cs file to test CS of this repository

Discussion
----------

Add .php_cs file to test CS of this repository

I ignored most of the important files that don't need to be checked. Additionally, some test files need to be ignored.

So this means running a command like this in the root dir to check the CS of this repo:

```
php ./php-cs-fixer fix --level=all .
```

Note: I've not used the Finder component much, so happy to hear any best practice suggestions here.

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

by jalliot at 2012-05-22T15:29:19Z

@fabpot I think this tool should not provide the specific configs for Symfony 2.[01]. Symfony should ship with the `.php_cs` file instead IMO. What do you think?

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

by fabpot at 2012-05-22T15:55:15Z

Being a tool that I created for Symfony first, let's keep these files in the main repo. And if other projects want theirs to be hosted here too, I'm fine with that.
fixed CS
Magento project config and finder
This allows a full PSR-2 analysis of a magento ecommerce project.  It
will skip any package files (core, enterprise, or xml) to allow
upgradability and patching.
merged branch ajgarlag/master (PR #31)

Commits
-------

8ed92c6 Relax php version dependency to PHP 5.3.3 so it could be used with Debian Squeeze

Discussion
----------

Relax php version dependency

I was trying to test the php-cs-fixer, but I cannot run it because of php version.

I'm running Debian Squeeze (stable) which includes PHP 5.3.3, so I've pached the source to use an alternative method for finding file extension when running a PHP version lower than 5.3.6. This method is suggested in http://www.php.net/manual/en/splfileinfo.getextension.php#refsect1-splfileinfo.getextension-notes

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

by fabpot at 2012-05-22T15:53:29Z

That's perfect. Now, before I merge, I'm going to ask you to squash your commit. You can find information on how to do that here: http://symfony.com/doc/current/contributing/code/patches.html#rework-your-patch

Thanks.

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

by ajgarlag at 2012-05-22T16:24:03Z

I've squashed the commit (thanks for the hint). I'll make another pull request with the travis-ci config file
merged branch mykehsd/master (PR #37)

Commits
-------

31b258f Magento project config and finder This allows a full PSR-2 analysis of a magento ecommerce project.  It will skip any package files (core, enterprise, or xml) to allow upgradability and patching.

Discussion
----------

Magento Ecommerce finder and config

This allows a full PSR-2 analysis of a magento ecommerce project.  It
will skip any package files (core, enterprise, or xml) to allow
upgradability and patching.
updated README and phar
Adds travis-ci config file