Conversation
|
Please improve your commit message according to the contributing guide. In the present case, I have no idea why you would want to move psalm.xml to psalm.xml.dist |
b93514e to
04653e0
Compare
|
@greg0ire I considered this as self-explanatory, sorry 🤷♂️ I updated the commit message accordingly. |
|
No worries. Why would a user want to override |
|
We are doing the same for |
|
I can think of some reasons to do it for phpunit.xml on some repositories, but I'd be really interested to know how many people actually resort to it, or why you would need to do it in the case of Psalm. I think it only complicates things as one might forget they once did it and wonder why they get different results locally or in the CI. Personally, the very rare times I've had to use a different database, I've instead run |
|
The only reason I could think of regarding Psalm is lowering the level locally to see the issues that should probably targeted in a next major. Anyways, I don't care that much, my idea was rather to have all build tools (i.e. PhpCS, PHPUnit, PHPStan and Psalm) consistent in terms of their configuration. |
04653e0 to
678021e
Compare
|
@greg0ire I updated the commit, let's keep |
No description provided.