phpmyadmin / error-reporting-server

phpMyAdmin server side component for the error reporting system
MIT License
19 stars 28 forks source link

Move to more phpMyAdmin-like coding style #131

Closed devenbansod closed 7 years ago

devenbansod commented 7 years ago

Signed-off-by: Deven Bansod devenbansod.bits@gmail.com

codecov[bot] commented 7 years ago

Codecov Report

Merging #131 into master will decrease coverage by 1.49%. The diff coverage is 53.65%.

@@             Coverage Diff             @@
##             master     #131     +/-   ##
===========================================
- Coverage     53.92%   52.43%   -1.5%     
  Complexity      212      212             
===========================================
  Files            17       17             
  Lines           942      967     +25     
===========================================
- Hits            508      507      -1     
- Misses          434      460     +26
nijel commented 7 years ago

Thanks! Are you doing the changes manually or have you configured php-cs-fixer? The latter might be better approach in the long run (though I'm not sure how much it can fix).

devenbansod commented 7 years ago

I currently did part-manually and part by phpcbf with original coding-standard. But I will look into configuring the tool you mentioned. If it works, then that'd be great!

nijel commented 7 years ago

We're already using it for motranslator, shapefile and sql-parser, so you can base on the configuration there (though that is much closer to PSR than what we use at phpMyAdmin).

devenbansod commented 7 years ago

Thanks for the pointer. I have made a new PR at #132 with the auto-fixes done by php-cs-fixer and have added a simiar config file to this repo (with changes to the folders to be fixed).