{"action":"created","issue":{"url":"https://api.github.com/repos/laravel/framework/issues/9638","labels_url":"https://api.github.com/repos/laravel/framework/issues/9638/labels{/name}","comments_url":"https://api.github.com/repos/laravel/framework/issues/9638/comments","events_url":"https://api.github.com/repos/laravel/framework/issues/9638/events","html_url":"https://github.com/laravel/framework/pull/9638","id":95273368,"number":9638,"title":"[5.1] Improve artisan make:listener feedback","user":{"login":"Arrilot","id":2826480,"avatar_url":"https://avatars.githubusercontent.com/u/2826480?v=3","gravatar_id":"","url":"https://api.github.com/users/Arrilot","html_url":"https://github.com/Arrilot","followers_url":"https://api.github.com/users/Arrilot/followers","following_url":"https://api.github.com/users/Arrilot/following{/other_user}","gists_url":"https://api.github.com/users/Arrilot/gists{/gist_id}","starred_url":"https://api.github.com/users/Arrilot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Arrilot/subscriptions","organizations_url":"https://api.github.com/users/Arrilot/orgs","repos_url":"https://api.github.com/users/Arrilot/repos","events_url":"https://api.github.com/users/Arrilot/events{/privacy}","received_events_url":"https://api.github.com/users/Arrilot/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-07-15T19:39:05Z","updated_at":"2015-07-16T08:29:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/laravel/framework/pulls/9638","html_url":"https://github.com/laravel/framework/pull/9638","diff_url":"https://github.com/laravel/framework/pull/9638.diff","patch_url":"https://github.com/laravel/framework/pull/9638.patch"},"body":"Closes #9576\r\n\r\nAs you can see people tend to use this command without required `--event` option. \r\nI find the fact that the generator creates a broken listener class with syntax errors in this case pretty confusing."},"comment":{"url":"https://api.github.com/repos/laravel/framework/issues/comments/121876666","html_url":"https://github.com/laravel/framework/pull/9638#issuecomment-121876666","issue_url":"https://api.github.com/repos/laravel/framework/issues/9638","id":121876666,"user":{"login":"Arrilot","id":2826480,"avatar_url":"https://avatars.githubusercontent.com/u/2826480?v=3","gravatar_id":"","url":"https://api.github.com/users/Arrilot","html_url":"https://github.com/Arrilot","followers_url":"https://api.github.com/users/Arrilot/followers","following_url":"https://api.github.com/users/Arrilot/following{/other_user}","gists_url":"https://api.github.com/users/Arrilot/gists{/gist_id}","starred_url":"https://api.github.com/users/Arrilot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Arrilot/subscriptions","organizations_url":"https://api.github.com/users/Arrilot/orgs","repos_url":"https://api.github.com/users/Arrilot/repos","events_url":"https://api.github.com/users/Arrilot/events{/privacy}","received_events_url":"https://api.github.com/users/Arrilot/received_events","type":"User","site_admin":false},"created_at":"2015-07-16T08:29:19Z","updated_at":"2015-07-16T08:29:19Z","body":"@taylorotwell You can't really mark an option as required. Options are optional :)\r\nLook what symfony documentation says \"InputOption::VALUE_REQUIRED\tThis value is required (e.g. --iterations=5), the option itself is still optional\" http://symfony.com/doc/current/components/console/introduction.html\r\n\r\nOn the other hand the command is not designed to work without an event (Look at the stubs).\r\n\r\nSo, this change looks okay to me.\r\nDo you want a more general approach that can be reused in other commands? I'm not sure it's really needed here.\r\n\r\n"}}