Juliette

Results 2034 comments of Juliette

@joachim-n Thanks for opening this issue. The docs are actually correct, but I do agree this can be clarified some more. Basically it comes down to this: * For a...

> I hadn't realised that a Standard and a ruleset are two different things. Well, they are and they aren't 😅. That's what makes it complicated. For the ruleset file,...

Just realized I forgot to update you, but I have added a new page ["About Standards for PHP_CodeSniffer"](https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/About-Standards-for-PHP_CodeSniffer) to the wiki, which contains a short explanation about the difference between...

@joachim-n Thanks for reading though this. I'll try to answer your follow-up question, but AFAICS that wasn't part of the original question posed in this ticket. > It could maybe...

@joachim-n I agree, there's definitely a lot more documentation which would be helpful to be written. I'll keep the above in mind as one of those things ;-) I have...

Oh and before I forget: considering the new wiki page addresses the original issue, can we close this one ?

> Note: as things stand, testing against PHPCS 4.x requires to run composer install using --prefer-source as the abstract test case file is no longer included in the package for...

@VladaHejda Thank you for reporting this. This is due to the sniff incorrectly acting on short lists. The `Squiz.Arrays.ArrayDeclaration` sniff is [known to be problematic and has been for years](https://github.com/squizlabs/PHP_CodeSniffer/issues/582)....

> Initially, I thought the idea was to bail early when the `T_OPEN_SHORT_ARRAY` token represents a short list and not an array. But it seems that doing that without #12...

> Please assign [Soft-Deprecate __sleep() and __wakeup()](https://wiki.php.net/rfc/soft-deprecate-sleep-wakeup) to me. I'll tinker on my own as best I can and reach out if I have questions. @afilina Done. The reason I...