Juliette

@mikemanger Thanks for reporting this. We are aware of this, but to be honest, this is nothing to be concerned about. Deprecation notices are advance notices of upcoming changes and...

Just want to leave a note here that the workflow as described above presumes a Composer based project, while PHPCS is not limited to Composer supported projects.

Previous issue which may be loosely related: * https://github.com/squizlabs/PHP_CodeSniffer/issues/2965 which was fixed via https://github.com/squizlabs/PHP_CodeSniffer/commit/a3e045313cb7651b587acca17e190cbfd0c81fae

Also loosely related: PHPCS records an "Error opening file; file no longer exists or you do not have access to read the file" error when an unreadable file is encountered....

> Not sure what's wrong with the failing build :/ @gmponos As per: https://travis-ci.org/github/squizlabs/PHP_CodeSniffer/jobs/701538216#L544-L549 you didn't add the new files to the `package.xml` file which means they would not be...

> And foreach... https://3v4l.org/LHkDc Fair enough, but using a foreach without body doesn't really _do_ anything, so there wouldn't be any point doing that. The one exception being when using...

@gmponos This is not about braces or no braces. My example is 100% valid code without any mistakes. And just like you say, as it is now common to use...

At the very least, please consider using a different error code for with/without curly brace as that way, people who use the short notation without body could still use the...

https://github.com/PHPCSStandards/PHP_CodeSniffer/issues/1202 has brought this to my attention again. While adding the methods to PHPCSUtils would mean the PHPCS native sniffs can't use the methods (at this time), any sniff which...

PRs #719 and #720 are now open to address this. Reviews welcome.