use phpcs.xml.dist for scrutinizer

[mmoll]

No description provided.

[mmoll]

let's see where this is going, per https://scrutinizer-ci.com/docs/tools/php/code-sniffer/

[codecov]

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@ ## master #52 +/- ## ========================================= Coverage 96.08% 96.08% Complexity 122 122 ========================================= Files 6 6 Lines 639 639 ========================================= Hits 614 614 Misses 25 25

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c44dd9...75851ab. Read the comment docs.

[xalopp]

Maybe overriding the step with - phpcs-run --standard=phpcs.xml will work?

[mmoll]

I wiped the whole phpcs defaults of Scrutinizer, now it seems good.

[xalopp]

Sure? For me the log for the phpcs-run still looks the same. https://scrutinizer-ci.com/g/mayflower/mo4-coding-standard/inspections/84849830-413e-43b1-b9fd-13a175a7062e/log

Maye adding an error the phpcs.xml.dist will point out this file is used or not.

[mmoll]

The "Patches" tab in the report is gone now and there are no generated "recommended" patches anymore.

[xalopp]

Ok, now I get it 🤦‍♂️