{"action":"created","issue":{"url":"https://api.github.com/repos/openSUSE/docmanager/issues/8","labels_url":"https://api.github.com/repos/openSUSE/docmanager/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/openSUSE/docmanager/issues/8/comments","events_url":"https://api.github.com/repos/openSUSE/docmanager/issues/8/events","html_url":"https://github.com/openSUSE/docmanager/issues/8","id":70928525,"number":8,"title":"Omit the -f/--file option","user":{"login":"tomschr","id":1312925,"avatar_url":"https://avatars.githubusercontent.com/u/1312925?v=3","gravatar_id":"","url":"https://api.github.com/users/tomschr","html_url":"https://github.com/tomschr","followers_url":"https://api.github.com/users/tomschr/followers","following_url":"https://api.github.com/users/tomschr/following{/other_user}","gists_url":"https://api.github.com/users/tomschr/gists{/gist_id}","starred_url":"https://api.github.com/users/tomschr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomschr/subscriptions","organizations_url":"https://api.github.com/users/tomschr/orgs","repos_url":"https://api.github.com/users/tomschr/repos","events_url":"https://api.github.com/users/tomschr/events{/privacy}","received_events_url":"https://api.github.com/users/tomschr/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openSUSE/docmanager/labels/idea","name":"idea","color":"0052cc"},{"url":"https://api.github.com/repos/openSUSE/docmanager/labels/question","name":"question","color":"cc317c"}],"state":"open","locked":false,"assignee":{"login":"rsalevsky","id":4128547,"avatar_url":"https://avatars.githubusercontent.com/u/4128547?v=3","gravatar_id":"","url":"https://api.github.com/users/rsalevsky","html_url":"https://github.com/rsalevsky","followers_url":"https://api.github.com/users/rsalevsky/followers","following_url":"https://api.github.com/users/rsalevsky/following{/other_user}","gists_url":"https://api.github.com/users/rsalevsky/gists{/gist_id}","starred_url":"https://api.github.com/users/rsalevsky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rsalevsky/subscriptions","organizations_url":"https://api.github.com/users/rsalevsky/orgs","repos_url":"https://api.github.com/users/rsalevsky/repos","events_url":"https://api.github.com/users/rsalevsky/events{/privacy}","received_events_url":"https://api.github.com/users/rsalevsky/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-04-25T15:41:04Z","updated_at":"2015-05-04T07:42:39Z","closed_at":null,"body":"According to the README and `--help`, `docmanager` needs a `-f` or `--files` option. IMHO this is unnecessary and should be removed.\r\n\r\nFor example, `git`, `svn`, and many other commands usually don't have a `-f` option to operate on their file list. They just use it without such an option. The requirement of `-f` looks \"unnatural\" to me.\r\n\r\nFor that reason, I would suggest to change this code in `__init__.py`:\r\n\r\n```diff\r\ndiff --git i/docmanager/__init__.py w/docmanager/__init__.py                                                                 \r\nindex a4047bf..9f0580a 100644\r\n--- i/docmanager/__init__.py\r\n+++ w/docmanager/__init__.py\r\n@@ -37,12 +37,8 @@ class ArgParser:\r\n         self.parse_arguments()\r\n \r\n     def add_arguments(self):\r\n-        file_group = self.__parser.add_argument_group(\"Files\")\r\n-        file_group.add_argument(\r\n-                    \"-f\",\r\n-                    \"--files\",\r\n-                    nargs=\"+\",\r\n-                    required=True,\r\n+        self.__parser.add_argument(\"FILES\",\r\n+                    nargs=\"+\",\r\n+                    dest=\"files\",\r\n                     help=\"One or more DocBook XML or DC files.\"\r\n                 )\r\n```\r\n\r\nWith that change, you can use `docmanager` like this:\r\n\r\n```shell\r\ndocmanager --set maintainer=test_name version=test_version test_file1.xml test_file2.xml\r\n```\r\nWhat do you think? Is there any special reason to use `-f`?"},"comment":{"url":"https://api.github.com/repos/openSUSE/docmanager/issues/comments/98612148","html_url":"https://github.com/openSUSE/docmanager/issues/8#issuecomment-98612148","issue_url":"https://api.github.com/repos/openSUSE/docmanager/issues/8","id":98612148,"user":{"login":"rsalevsky","id":4128547,"avatar_url":"https://avatars.githubusercontent.com/u/4128547?v=3","gravatar_id":"","url":"https://api.github.com/users/rsalevsky","html_url":"https://github.com/rsalevsky","followers_url":"https://api.github.com/users/rsalevsky/followers","following_url":"https://api.github.com/users/rsalevsky/following{/other_user}","gists_url":"https://api.github.com/users/rsalevsky/gists{/gist_id}","starred_url":"https://api.github.com/users/rsalevsky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rsalevsky/subscriptions","organizations_url":"https://api.github.com/users/rsalevsky/orgs","repos_url":"https://api.github.com/users/rsalevsky/repos","events_url":"https://api.github.com/users/rsalevsky/events{/privacy}","received_events_url":"https://api.github.com/users/rsalevsky/received_events","type":"User","site_admin":false},"created_at":"2015-05-04T07:42:39Z","updated_at":"2015-05-04T07:42:39Z","body":"I agree, you can test it and when there are no problems then I'm fine with this idea. "}}