{"action":"created","issue":{"url":"https://api.github.com/repos/holgerbrandl/kscript/issues/112","repository_url":"https://api.github.com/repos/holgerbrandl/kscript","labels_url":"https://api.github.com/repos/holgerbrandl/kscript/issues/112/labels{/name}","comments_url":"https://api.github.com/repos/holgerbrandl/kscript/issues/112/comments","events_url":"https://api.github.com/repos/holgerbrandl/kscript/issues/112/events","html_url":"https://github.com/holgerbrandl/kscript/issues/112","id":307815173,"number":112,"title":"Suggestion about text mode","user":{"login":"NightMachinary","id":36224762,"avatar_url":"https://avatars1.githubusercontent.com/u/36224762?v=4","gravatar_id":"","url":"https://api.github.com/users/NightMachinary","html_url":"https://github.com/NightMachinary","followers_url":"https://api.github.com/users/NightMachinary/followers","following_url":"https://api.github.com/users/NightMachinary/following{/other_user}","gists_url":"https://api.github.com/users/NightMachinary/gists{/gist_id}","starred_url":"https://api.github.com/users/NightMachinary/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NightMachinary/subscriptions","organizations_url":"https://api.github.com/users/NightMachinary/orgs","repos_url":"https://api.github.com/users/NightMachinary/repos","events_url":"https://api.github.com/users/NightMachinary/events{/privacy}","received_events_url":"https://api.github.com/users/NightMachinary/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-22T21:07:49Z","updated_at":"2018-03-22T21:28:27Z","closed_at":null,"author_association":"NONE","body":"Make \"-t\" get the version of API as an argument, use semantic versioning for the API, and interpret no argument and zero as the latest version. Or make another flag -tv which accepts the version and reserve -t for the latest version."},"comment":{"url":"https://api.github.com/repos/holgerbrandl/kscript/issues/comments/375463575","html_url":"https://github.com/holgerbrandl/kscript/issues/112#issuecomment-375463575","issue_url":"https://api.github.com/repos/holgerbrandl/kscript/issues/112","id":375463575,"user":{"login":"holgerbrandl","id":200952,"avatar_url":"https://avatars2.githubusercontent.com/u/200952?v=4","gravatar_id":"","url":"https://api.github.com/users/holgerbrandl","html_url":"https://github.com/holgerbrandl","followers_url":"https://api.github.com/users/holgerbrandl/followers","following_url":"https://api.github.com/users/holgerbrandl/following{/other_user}","gists_url":"https://api.github.com/users/holgerbrandl/gists{/gist_id}","starred_url":"https://api.github.com/users/holgerbrandl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/holgerbrandl/subscriptions","organizations_url":"https://api.github.com/users/holgerbrandl/orgs","repos_url":"https://api.github.com/users/holgerbrandl/repos","events_url":"https://api.github.com/users/holgerbrandl/events{/privacy}","received_events_url":"https://api.github.com/users/holgerbrandl/received_events","type":"User","site_admin":false},"created_at":"2018-03-22T21:28:27Z","updated_at":"2018-03-22T21:28:27Z","author_association":"OWNER","body":"I see the point.\r\n\r\nUntil now I was hoping to rather keep the text support API stable after some initial time of maturation. So far it worked for me as it is, but I have not received any external feedback. \r\n\r\nIt's a tradeoff of CLI complexity/simplicity and long-term reproducibility.\r\n\r\nIf you doubt the long-term stability, you could also just add the 3 lines it is injecting, manually to your script. But it would be clearly a bit more verbose and not a one-liner anymore.\r\n\r\nFinally, from a user perspective, if the -t without argument would refer to the latest version, most users would go for it, without bothering to lookup the current version of the support API I guess. Making the version a mandatory argument, would be tedious and confusing for the user.\r\n\r\nI'd suggest to implement the fix, the first time we roll out a backwards incompatible change to the text support API. What do you think?\r\n\r\n"}}
{"actor":{"display_login":"holgerbrandl"}}