Laurent Laville

Results 239 comments of Laurent Laville

I think it's a shame not to be able to use the autofix feature for PHP Linters. So, even if PHP-CS-Fixer new linter is ready, I'm still working to be...

@nvuillam I need your help to finish this linter. Following https://megalinter.io/latest/contributing/#add-a-new-linter I don't know what is missing ! see chunk of deployment/test workflow ``` =========================== short test summary info ============================...

@nvuillam After trying your tip, and see that it won't fix anything (sorry to said that), I've spent my time to track down where it came from. So I've look...

@bdovaz There is no more hurry now. I'll be slow to respond in next days because I'll have a slow internet connection !

@nvuillam You can help, either : - by replying to my comment (https://github.com/oxsecurity/megalinter/issues/3550#issuecomment-2119666471) on quick patch - or suggest a new approach and better fix ! Because PHP-CS-Fixer run ok...

Will have to wait I return to my home with a better internet connection. I'm very limited now :(

@nvuillam no sorry, make your release without it !

@nvuillam I've finally a good new ! I've fixed my mobile internet connection, and I should be able to prepare a PR today

I'm on way to prepare a PR. Just testing PHP flavor rebuilt with current branch rebased with main

Consider the PR 3598 as a preview ! I don't like the quick fix https://github.com/oxsecurity/megalinter/commit/db504eddb0b0f73a49b5bb33f4271e391cb0da1b but at least unit tests PASSED (locally ... wait for CI results now)