{"action":"created","issue":{"url":"https://api.github.com/repos/fish-shell/fish-shell/issues/1511","repository_url":"https://api.github.com/repos/fish-shell/fish-shell","labels_url":"https://api.github.com/repos/fish-shell/fish-shell/issues/1511/labels{/name}","comments_url":"https://api.github.com/repos/fish-shell/fish-shell/issues/1511/comments","events_url":"https://api.github.com/repos/fish-shell/fish-shell/issues/1511/events","html_url":"https://github.com/fish-shell/fish-shell/issues/1511","id":36068299,"number":1511,"title":"Validate arguments to rm command","user":{"login":"siteshwar","id":1300760,"avatar_url":"https://avatars.githubusercontent.com/u/1300760?v=3","gravatar_id":"","url":"https://api.github.com/users/siteshwar","html_url":"https://github.com/siteshwar","followers_url":"https://api.github.com/users/siteshwar/followers","following_url":"https://api.github.com/users/siteshwar/following{/other_user}","gists_url":"https://api.github.com/users/siteshwar/gists{/gist_id}","starred_url":"https://api.github.com/users/siteshwar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/siteshwar/subscriptions","organizations_url":"https://api.github.com/users/siteshwar/orgs","repos_url":"https://api.github.com/users/siteshwar/repos","events_url":"https://api.github.com/users/siteshwar/events{/privacy}","received_events_url":"https://api.github.com/users/siteshwar/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/fish-shell/fish-shell/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/fish-shell/fish-shell/milestones/4","html_url":"https://github.com/fish-shell/fish-shell/milestones/fish-future","labels_url":"https://api.github.com/repos/fish-shell/fish-shell/milestones/4/labels","id":130257,"number":4,"title":"fish-future","description":"These are bugs that are not targeted for an upcoming release. After a release, we will reexamine these bugs and pull any in that we think are good targets.\r\n\r\nIf you can provide a fix for a bug in this milestone, we'll generally accept it for the next release even if the bug is not in the release.","creator":{"login":"ridiculousfish","id":920838,"avatar_url":"https://avatars.githubusercontent.com/u/920838?v=3","gravatar_id":"","url":"https://api.github.com/users/ridiculousfish","html_url":"https://github.com/ridiculousfish","followers_url":"https://api.github.com/users/ridiculousfish/followers","following_url":"https://api.github.com/users/ridiculousfish/following{/other_user}","gists_url":"https://api.github.com/users/ridiculousfish/gists{/gist_id}","starred_url":"https://api.github.com/users/ridiculousfish/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ridiculousfish/subscriptions","organizations_url":"https://api.github.com/users/ridiculousfish/orgs","repos_url":"https://api.github.com/users/ridiculousfish/repos","events_url":"https://api.github.com/users/ridiculousfish/events{/privacy}","received_events_url":"https://api.github.com/users/ridiculousfish/received_events","type":"User","site_admin":false},"open_issues":371,"closed_issues":6,"state":"open","created_at":"2012-06-11T02:25:13Z","updated_at":"2016-03-07T12:06:49Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2014-06-19T10:58:24Z","updated_at":"2016-03-07T18:03:23Z","closed_at":null,"body":"Arguments to rm command should be validated to avoid accidentally deleting important  files. Commands like :\r\n```\r\nrm -rf *\r\n```\r\nshould be confirmed from the user. \r\n\r\nOn GNU systems we can pass ```-I``` parameter to rm command to get confirmation from user before deleting more than 3 files."},"comment":{"url":"https://api.github.com/repos/fish-shell/fish-shell/issues/comments/193373042","html_url":"https://github.com/fish-shell/fish-shell/issues/1511#issuecomment-193373042","issue_url":"https://api.github.com/repos/fish-shell/fish-shell/issues/1511","id":193373042,"user":{"login":"krader1961","id":7545917,"avatar_url":"https://avatars.githubusercontent.com/u/7545917?v=3","gravatar_id":"","url":"https://api.github.com/users/krader1961","html_url":"https://github.com/krader1961","followers_url":"https://api.github.com/users/krader1961/followers","following_url":"https://api.github.com/users/krader1961/following{/other_user}","gists_url":"https://api.github.com/users/krader1961/gists{/gist_id}","starred_url":"https://api.github.com/users/krader1961/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krader1961/subscriptions","organizations_url":"https://api.github.com/users/krader1961/orgs","repos_url":"https://api.github.com/users/krader1961/repos","events_url":"https://api.github.com/users/krader1961/events{/privacy}","received_events_url":"https://api.github.com/users/krader1961/received_events","type":"User","site_admin":false},"created_at":"2016-03-07T18:03:23Z","updated_at":"2016-03-07T18:03:23Z","body":"I'm strongly inclined to reject this request.  Is there anyone who believes that fish should support this natively? If so I'd ask for an outline of a solution that isn't a stupid, easily fooled, hack like zsh and bash employ. For example, should there be a mechanism, perhaps involving the `commandline` builtin, by which a function could determine whether globs (not just `*`) were present in the original statement. Or should the `function` command have a new option that registers its interest in being passed the raw globs without expansion? It could then use the hypothetical `glob` command being discussed in another issue to explicitly expand them."}}