silverstripe / silverstripe-reports

API for creating backend reports in the Silverstripe Framework.
BSD 3-Clause "New" or "Revised" License
6 stars 28 forks source link

Rescue Master Branch PR: PSR12 compliance #154

Closed GuySartorelli closed 2 years ago

GuySartorelli commented 2 years ago

Rescues https://github.com/silverstripe/silverstripe-reports/pull/115

Targetting 4 since there are no API changes. This allows both major lines to benefit and reduces merge-up pain. Excludes the swap from code/ to src/ to reduce merge-up pain - there isn't really much (if any) benefit to making that change. If someone reviewing this does want that change included, I will do it as a separate PR targetting 5 just to be safe.

Parent issue