leroy-merlin-br / larasniffer Goto Github PK

Hi, mate!

Just a quick issue I got when installing the package on a fresh Laravel installation. When trying to do php artisan sniff I get the following error:

Undefined property: LeroyMerlin\LaraSniffer\SniffCommand::$stream

I've added the ServiceProvider to the app.php config, also published the configuration.

Thanks for your help.

Nowadays PHPSniffer phpcs.xml configuration can:

• Configure colored outputs
• Set codestyle rules
• Configurable include and exclude paths patterns

Since this package does not add anything new to default code sniffer and does not support Laravel 5 too, I think it is prudent to abandon this package and document how to do the same using only PHP Sniffer.

Any toughs? @Zizaco , @GuilhermeGuitte , @ravanscafi , @rplansky ?

Can u bump the version plz.

wouldn't it be nice to exclude *.blade.php files in generally or to have an exclude list for sub folders? Would be easier to add the /app/*.php files then without getting the views into the sniffer.

How can i make an exception rules ? Like for spacing, tabs and newlines? I think, this condition can be ignored

Right now its all or nothing kind of situation. I'd love to have PSR-2 through the entire app, besides a few "hard to comply" areas, such as database migrations, views, etc, which could fall under a less strict ruleset. Is that possible? Would that be a difficult modification?