{"action":"created","issue":{"url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/1169","repository_url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer","labels_url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/1169/labels{/name}","comments_url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/1169/comments","events_url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/1169/events","html_url":"https://github.com/squizlabs/PHP_CodeSniffer/issues/1169","id":178666798,"number":1169,"title":"Sniff unit tests fail because of the <exclude-pattern> behaviour","user":{"login":"PatrickRose","id":1681612,"avatar_url":"https://avatars.githubusercontent.com/u/1681612?v=3","gravatar_id":"","url":"https://api.github.com/users/PatrickRose","html_url":"https://github.com/PatrickRose","followers_url":"https://api.github.com/users/PatrickRose/followers","following_url":"https://api.github.com/users/PatrickRose/following{/other_user}","gists_url":"https://api.github.com/users/PatrickRose/gists{/gist_id}","starred_url":"https://api.github.com/users/PatrickRose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PatrickRose/subscriptions","organizations_url":"https://api.github.com/users/PatrickRose/orgs","repos_url":"https://api.github.com/users/PatrickRose/repos","events_url":"https://api.github.com/users/PatrickRose/events{/privacy}","received_events_url":"https://api.github.com/users/PatrickRose/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-09-22T17:04:42Z","updated_at":"2016-12-04T17:52:14Z","closed_at":"2016-09-23T01:02:31Z","body":"I have a sniff set up like so in my standard's ruleset.xml\n\n``` xml\n<rule ref=\"StandardName.PHP.SniffName\">\n    <!--Have to use a negative lookahead as PHPCS doesn't have support for include patterns-->\n    <exclude-pattern>path/that/includes/phpcs/directory</exclude-pattern>\n</rule>\n```\n\nWhen I run the Sniff unit tests, the `.inc` files for testing are being excluded so my unit tests fail.\n"},"comment":{"url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/comments/264718868","html_url":"https://github.com/squizlabs/PHP_CodeSniffer/issues/1169#issuecomment-264718868","issue_url":"https://api.github.com/repos/squizlabs/PHP_CodeSniffer/issues/1169","id":264718868,"user":{"login":"aik099","id":1277526,"avatar_url":"https://avatars.githubusercontent.com/u/1277526?v=3","gravatar_id":"","url":"https://api.github.com/users/aik099","html_url":"https://github.com/aik099","followers_url":"https://api.github.com/users/aik099/followers","following_url":"https://api.github.com/users/aik099/following{/other_user}","gists_url":"https://api.github.com/users/aik099/gists{/gist_id}","starred_url":"https://api.github.com/users/aik099/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aik099/subscriptions","organizations_url":"https://api.github.com/users/aik099/orgs","repos_url":"https://api.github.com/users/aik099/repos","events_url":"https://api.github.com/users/aik099/events{/privacy}","received_events_url":"https://api.github.com/users/aik099/received_events","type":"User","site_admin":false},"created_at":"2016-12-04T17:52:14Z","updated_at":"2016-12-04T17:52:14Z","body":"A nasty side effect of this fix is that unit test runner now only looks at sniffs from standards listed in `installed_paths` config setting hence #1205 problem.\r\n\r\nSomehow the `installed_paths` setting by default points to standards folder of PHP_CodeSniffer itself and that's why sniff discovery isn't broken in there. All custom test suites are out of luck however."}}
{"actor":{"display_login":"aik099"}}