{"action":"opened","issue":{"url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/702","repository_url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards","labels_url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/702/labels{/name}","comments_url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/702/comments","events_url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/702/events","html_url":"https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/702","id":181000921,"number":702,"title":"phpcbf adds long conditional end comments without leading space","user":{"login":"JDGrimes","id":4005415,"avatar_url":"https://avatars.githubusercontent.com/u/4005415?v=3","gravatar_id":"","url":"https://api.github.com/users/JDGrimes","html_url":"https://github.com/JDGrimes","followers_url":"https://api.github.com/users/JDGrimes/followers","following_url":"https://api.github.com/users/JDGrimes/following{/other_user}","gists_url":"https://api.github.com/users/JDGrimes/gists{/gist_id}","starred_url":"https://api.github.com/users/JDGrimes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JDGrimes/subscriptions","organizations_url":"https://api.github.com/users/JDGrimes/orgs","repos_url":"https://api.github.com/users/JDGrimes/repos","events_url":"https://api.github.com/users/JDGrimes/events{/privacy}","received_events_url":"https://api.github.com/users/JDGrimes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/labels/Component:%20Upstream","name":"Component: Upstream","color":"c5def5"},{"url":"https://api.github.com/repos/WordPress-Coding-Standards/WordPress-Coding-Standards/labels/Type:%20Bug","name":"Type: Bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-10-04T20:48:55Z","updated_at":"2016-10-04T20:48:55Z","closed_at":null,"body":"Given the code:\r\n\r\n```php\r\nif ( $something ) {\r\n    /* long codes */\r\n}\r\n```\r\n\r\nThe expected fix is:\r\n\r\n```php\r\nif ( $something ) {\r\n    /* long codes */\r\n} // End if().\r\n```\r\n\r\nThe actual result:\r\n\r\n```php\r\nif ( $something ) {\r\n    /* long codes */\r\n}// End if().\r\n```\r\n\r\nIntroduced in #653. Changing the `commentFormat` to include a leading space appears to have no effect. The sniff being used is `Squiz.Commenting.LongConditionClosingComment`, and it is designed to not include any space before the comment. We actually have to exclude an error that it produces when space is present. I'm not sure that we can fix this without getting the upstream sniff modified."}}
{"actor":{"display_login":"JDGrimes"}}