Juliette

Results 2039 comments of Juliette

Another one to add to the "catches me out/contrary to industry standard" list: * No spaces around the `=` sign for parameters with default values. One for a later point...

I've just opened PR #225 for yet another annoyance, but one which is serious enough that it decreases productivity. See the PR description for the reasoning.

Suggested by @fredden : > consider adding `Squiz.Commenting.FunctionComment.ParamCommentNotCapital` to `phpcs.xml.dist`

@fredden While I understand this is related to the other PR, I can't actually seem to create a (test) situation where I can reproduce an incorrect fixable count, let alone...

> @jrfnl I have written some steps to reproduce the problem and added these to the pull request description. When writing these, I realised that the problem is much larger...

@fredden and me discussed this PR in a call. Basically, this PR is blocked until the cache feature has proper (full) test coverage, so we can be sure nothing breaks...

@iVegas Thank you for your interest in contributing to PHP_CodeSniffer. There have been requests for this before. I suggest you read the previous discussion about this and take note of...

> Can we update the https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/Reporting page to describe how to set to use the custom report format for phpcs call? @iVegas That sounds like a good idea! Would you...

@Rasmus-Bertell This is not a bug, but by design. Hidden files are not checked. Also see: https://github.com/squizlabs/PHP_CodeSniffer/issues/1928 Considering this is only the second time this has comes up in 6...

Loosely related: https://github.com/squizlabs/PHP_CodeSniffer/issues/2916 (checking of files without extension, which will be included in v 4.0).