Cleanup coding standards

salsadigitalauorg / merlin-framework

Merlin - migration framework

GNU General Public License v3.0

17 stars 3 forks source link

Cleanup coding standards #161

Open AlexSkrypnyk opened 1 year ago

AlexSkrypnyk commented 1 year ago

The project uses mixed coding standards:

some files have an indentation of 2 spaces, some 4
vertical spacing is not followed consistently
comments formatting is not followed consistently
CI does not not scan tests dir

Is it possible to utilise Drupal or Symfony2 standard for this project (having a completely custom standard with 49 rules is a bit hard to follow :) ).

I'm happy to put a PR together to get this sorted.

steveworley commented 1 year ago

Agreed, I think it makes sense to standardise on the Symfony2 standards.